Datetime format string error during update loop

One of the update loop iterations is failing on a bad date string:
cosmos-upgrades Completed fetch data for network rsprovidertestnet                                                                                │
│ cosmos-upgrades Completed fetch data for network persistencetestnet2                                                                                │
│ cosmos-upgrades Completed fetch data for network sixtestnet                                                                                │
│ cosmos-upgrades Found 1 rest endpoints and 1 rpc endpoints for permtestnet                                                                                │
│ cosmos-upgrades <IP_ADDRESS> - - [19/Oct/2023 17:44:11] "GET /healthz HTTP/1.1" 200 -                                                                                │
│ cosmos-upgrades Completed fetch data for network permtestnet                                                                                │
│ cosmos-upgrades Traceback (most recent call last):                                                                                │
│ cosmos-upgrades   File "/app/app.py", line 703, in update_data                                                                                │
│ cosmos-upgrades     testnet_data = list(                                                                                │
│ cosmos-upgrades   File "/usr/local/lib/python3.9/concurrent/futures/_base.py", line 609, in result_iterator                                                              │
│ cosmos-upgrades     yield fs.pop().result()                                                                                │
│ cosmos-upgrades   File "/usr/local/lib/python3.9/concurrent/futures/_base.py", line 439, in result                                                                       │
│ cosmos-upgrades     return self.__get_result()                                                                                │
│ cosmos-upgrades   File "/usr/local/lib/python3.9/concurrent/futures/_base.py", line 391, in __get_result                                                                 │
│ cosmos-upgrades     raise self._exception                                                                                │
│ cosmos-upgrades   File "/usr/local/lib/python3.9/concurrent/futures/thread.py", line 58, in run                                                                          │
│ cosmos-upgrades     result = self.fn(*self.args, **self.kwargs)                                                                                │
│ cosmos-upgrades   File "/app/app.py", line 707, in <lambda>                                                                                │
│ cosmos-upgrades     lambda network, path: fetch_data_for_network(                                                                                │
│ cosmos-upgrades   File "/app/app.py", line 623, in fetch_data_for_network                                                                                │
│ cosmos-upgrades     current_block_datetime = parse_isoformat_string(current_block_time)                                                                                │
│ cosmos-upgrades   File "/app/app.py", line 207, in parse_isoformat_string                                                                                │
│ cosmos-upgrades     return datetime.fromisoformat(date_string)                                                                                │
│ cosmos-upgrades Error in update_data loop after 95.593695 seconds: Invalid isoformat string: '2023-10-19T17:43:49.9951+00:00'                                            │
│ cosmos-upgrades Error encountered. Sleeping for 1 minute before retrying...                                                                                │
│ cosmos-upgrades ValueError: Invalid isoformat string: '2023-10-19T17:43:49.9951+00:00'                                                                                │
│ cosmos-upgrades <IP_ADDRESS> - - [19/Oct/2023 17:44:12] "GET /healthz HTTP/1.1" 200 -

We need to catch the error and/or parse the string differently depending on the issue.

The error is here:
https://github.com/DefiantLabs/cosmos-upgrades/blob/b595fed83bd47f840f568a7d2699586d5fa36162/app.py#L618C18-L626
This means that the RPC endpoint is giving back a date that is not ISO format. I will need to figure out which network is throwing the error and what date format it is returning.

This actually has to do with the time returned by the RPC endpoint having a variable number of milliseconds.
We had anticipated this in our date parser function here:
https://github.com/DefiantLabs/cosmos-upgrades/blob/b595fed83bd47f840f568a7d2699586d5fa36162/app.py#L204-L207
However, the regex only looks for times that have MORE than 6 levels of precision, and the error is indicating 5 levels of precision: 2023-10-19T17:43:49.9951+00:00.
I will try to just turn the regex into a string split + add/remove levels of precision to get to the required 6 levels of precision for ISO.


Dev server doesn't update overridden env var when doing an automatic reload.

Describe the bug
I have a production API url and a local API URL I am trying to put into .env.local or .env.development. But after upgrading from vite 4.5.3 to 5.3.1, I noticed the dev server is not applying env vars correctly when doing an automatic reload.
Note that the frontend shows the correct value when killing the dev server and manually re-running from terminal
Reproduction
https://stackblitz.com/edit/vitejs-vite-y1ptn5?file=main.js,.env,.env.development&terminal=dev
Steps to reproduce

Put VITE_PUBLIC_API_URL=https://api.mysite.com in .env.
Add VITE_PUBLIC_API_URL=http://localhost:3100 in .env.local. After refreshing the frontend console.log(import.meta.env.VITE_PUBLIC_API_URL); shows the https://api.mysite.com value.
Do the same thing in .env.development, deleting .env.local, frontend still shows the old value

In StackBlitz you may have to delete .env.development, manually restart the server and then re-add it to reproduce.

System Info
System:
    OS: Linux 5.15 Ubuntu 22.04.4 LTS 22.04.4 LTS (Jammy Jellyfish)
    CPU: (16) x64 AMD Ryzen 7 3800X 8-Core Processor
    Memory: 11.82 GB / 15.56 GB
    Container: Yes
    Shell: 3.7.0 - /usr/bin/fish
  Binaries:
    Node: 18.14.0 - ~/.volta/tools/image/node/18.14.0/bin/node
    Yarn: 3.6.1 - ~/.volta/tools/image/yarn/1.22.19/bin/yarn
    npm: 9.3.1 - ~/.volta/tools/image/node/18.14.0/bin/npm
    pnpm: 8.6.5 - ~/.local/share/pnpm/pnpm
    bun: 0.5.8 - ~/.volta/bin/bun
    Watchman: 2<IP_ADDRESS> - /home/linuxbrew/.linuxbrew/bin/watchman
  Browsers:
    Chrome: 123.0.6312.105
    Chromium: 126.0.6478.61

Used Package Manager
pnpm
Logs
3:41:26 AM [vite] .env.local changed, restarting server...
3:41:26 AM [vite] server restarted.
3:42:06 AM [vite] .env.development changed, restarting server...
3:42:06 AM [vite] server restarted.
3:42:16 AM [vite] .env changed, restarting server...

Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

I wasn't able to reproduce on Stackblitz nor locally (Windows). Would you take a screen record so that I can try the exact same steps?

Sure, I'm traveling for the next two weeks without a computer but can get it for you in early January!


Segmentation fault (core dumped) when start velocyto

Hi,
I received the following error message when I type velocyto --help

I checked the issue #53 then I modify the setup.py file, but I got another error message in below:
Segmentation fault (core dumped)

This is a compiler error. First time I see this. I don't know how to solve
it. Maybe start a conda environment from scratch and try the installation
again?

Thank you for the prompt reply. I will have a try from scratch.

Hi,
When I run velocyto --help, I see Segmentation fault (core dumped) error. How can I solve the problem?

Hello gioelelm,
I have solved the issue, needed to install gcc-5 with the command conda install -c psi4 gcc-5 and then conda install -c anaconda libgcc

Great!

Hello gioelelm,
After I updated my conda I am again getting the Segmentation fault (core dumped) error. The solution that I used previously is not working now. Can you please suggest me how to fix it.


Rose Bush Drops

Rose bushes don't drop rose bush DOUBLE_PLANT when destroyed.

Fixed in the referenced PR.


Need docs and samples for Path.GetFullPath (string path, string basePath)

The current docs for Path.GetFullPath (string path, string basePath) are blank and only the signature appears. We need to fill out the docs with how to use the method, what each parameter is/does, and some samples.

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

ID: 2b6bcce3-8ba6-fed5-b1ad-da7442288f6d
Version Independent ID: eb2ecc15-c9fe-7fa9-d8d5-43dd0a431995
Content: Path.GetFullPath Method (System.IO)
Content Source: xml/System.IO/Path.xml
Product: dotnet-api
GitHub Login: @dotnet-bot
Microsoft Alias: dotnetcontent

This issue is addressed by dotnet/dotnet-api-docs#563.


I have questions about creating a pre-training model.

HI  We're working on a pre-training model. I have two questions about this process.
First of all, The amount of data I have is about 180 million sentences, and it takes too long to make a tfrecord. I need advice to make tfrecord.
Second, Is there no performance problem if I change the model type to another type when I create the Sentencepiece model? like bpe, char, or word.

If you happen to notice, there are two flags in the data_utils.py script that can be leveraged to perform paralleled preprocessing, namely FLAGS.num_task and FLAGS.task . Intuitively, num_task specifies the total number of processes you hope to use and FLAGS.task specifies the id of a process.
Hence, one (possible but not recommended) way to do this using bash alone could be
NUM_PROC=1000
For i in `seq 0 $((NUM_PROC - 1))`; do
  python data_utils.py \
    ....  \ # other flags
    --task=${i} \
    --num_task=${NUM_PROC} &
done

Essentially, you launch 1000 processes to process the data in parallel. For a better handling, you wrap the code with the python "multiprocessing" module.
For the second question, we haven't tried this yet. But my guess is it should be fine.

@zihangdai
I'll share the results the way you told me. Thank you so much.

@zihangdai
I have one more question.
Why is it that there is no "corpus_info_path" parameter when running "train.py"

corpus_info.json was originally design to keep track of the meta-information of the processed tfrecords. Potentially, one could use corpus_info.json to replace many data related flags. However, the currently release code does not take advantage of this and requires users to set them manually. This is a place we should improve.

Unless the --corpus_info_path option isn't used in train.py/train_gpu.py, we have to specify the same options in data_utils.py and train.py/train_gpu.py. So, it's better to set the same default parameters in these scripts. For example, the default parameters for --uncased are different.

Good catch. --corpus_info_path has been fixed. The other flags will be fixed soon.


Add activation layer to FeaturePyramidNetwork init

In some of my experiments, adding an activation layer to FPNs results in accuracy wins. More generally, it seems reasonable to allow it to be set like norm_layer.

Thanks for the PR @sklum . Do you have any example of popular models / papers that use a custom activation in the FPN?
If not, it might be best to leave it out as-is and let users copy/paste that class to edit it in their custom code (that's what a lot of these base classes were intended for.)


Add location picker to ObsEdit

Also gets GoogleMaps working, as long as you have the API key, which is needed for #396

@jtklein any idea why the e2e-Android test isn't passing here? I tried a couple different ways of exposing the GMAPS_API_KEY secret to AndroidManifest.xml, because that seemed to be the original problem.

I updated the "Create .env file" step to use printf to print the Google Maps key into the newly created .env file in the GitHub Action runner. It is using the same key as in env.example. It is building now, but I did not check if the actual key is used correctly. Since it is building we know that a value is present at not null, but it could be an empty string. Anyway, the emulator we are using in the runner has no Play Services enabled, so it will not be able to show the map as it is currently because it requires Play Services to be present on the device. At least that is what I see when I try to run the app on the same kind of emulator locally.


Add support for complex config options

This PR adds ability to use use custom loader options:
{
	module: {
		loaders: [{
			test: /\.css$/,
			loaders: [
				'style',
				'css?config=customCssConfig'
			]
		}]
	},
	customCssConfig: {}
}

Instead of #327, #315 I prefer to use recommended way to retrieve the loader config loader-utils#getloaderconfig.
@sokra Could you please review this PR?

Current coverage is 98.37% (diff: 100%)

Merging #348 into master will increase coverage by 0.02%

@@             master       #348   diff @@
==========================================
  Files             9         10     +1
  Lines           303        307     +4
  Methods          41         42     +1
  Messages          0          0
  Branches         66         67     +1
==========================================
+ Hits            298        302     +4
  Misses            5          5
  Partials          0          0

Powered by Codecov. Last update 22f6621...a0a43f6

cc @jhnns @bebraw

What's the benefit over options?

@bebraw Here is some of benefits:

it gives you ability to have shared config for group of xxx-loader
it gives you ability override default options without modifying query/options of  xxx-loader
it gives you ability to have custom configs for some user cases
it gives you ability to use recommended way to retrieve config of xxx-loader
etc

Please see:
https://github.com/webpack/less-loader#less-plugins
https://github.com/jtangelder/sass-loader#sass-options
https://github.com/webpack/html-loader#advanced-options
https://github.com/webpack/loader-utils/commit/7e86a18035633ffe041bfe43451e3e4352ee13c8

Gotcha. Looks like a good idea then so 👍 from me. I think @sokra can comment on the code once he has some time available.

@bebraw Thanks a lot.
It would be nice to have this feature as early as possible.
I'm looking forward to get feedback from @sokra.

I thought we agreed that polluting the webpack config object is not a good idea? We also recently added config object validation to webpack@2, so this will not work with webpack 2, right?

@jhnns Ok, I guess in that case we might to remove the feature from those other loaders to keep this consistent and document how to achieve the same effect without.

No longer allowed in webpack 2 as @jhnns said

it gives you ability to have shared config for group of xxx-loader

const sharedOptions = { ... }

it gives you ability override default options without modifying query/options of xxx-loader

options: {
  ...sharedOptions,
  key: value
}


Migrate confluence docs to README.

Migrated https://wiki.surfnet.nl/display/stepup/Middleware+Configuration

Nice to have in here :+1: (maybe time to split off into separate documents :yum:)


Add notification

Value Proposition
As a user, I want to get a notifications for my games so that don't miss them anymore.
Description

Acceptance Criteria

[ ] Display an overlay that show the game(s) of the current day

Tasks

[ ] Create new branch
[ ] Create new component for the notification
[ ] Add basic styling for the overlay
[ ] Create function for popup date === game.date

Looks convenient!
When will the pop-up open? Every time, the user opens the app on the respective day or only once at that day?


Unable to Use GPT-4

I'm unable to use GPT-4 with LMQL. The error message seems to suggest that lmql is hitting the wrong openai endpoint? If so, is there any way to specify the endpoint via lmql?
LMQL Prompt:
argmax
    """
    This is a test prompt. Please say something
    [OUTPUT]
    """
from
    "openai/gpt-4"
where

Error:
/home/pyodide/lmql/runtime/bopenai/batched_openai.py:681: OpenAIAPIWarning: OpenAI: ('This is a chat model and not supported in the v1/completions endpoint. Did you mean to use v1/chat/completions? (after receiving 0 chunks. Current chunk time: 0.7789998054504395 Average chunk time: 0.0)', 'Stream duration:', 0.7810001373291016) "<class 'lmql.runtime.bopenai.openai_api.OpenAIStreamError'>"
  warnings.warn("OpenAI: " + str(e) + ' "' + str(type(e)) + '"', category=OpenAIAPIWarning)
/home/pyodide/lmql/runtime/bopenai/batched_openai.py:669: OpenAIAPIWarning: Retrying 19 more times
  warnings.warn("Retrying {} more times".format(retries), category=OpenAIAPIWarning)
/home/pyodide/lmql/runtime/bopenai/batched_openai.py:681: OpenAIAPIWarning: OpenAI: ('This is a chat model and not supported in the v1/completions endpoint. Did you mean to use v1/chat/completions? (after receiving 0 chunks. Current chunk time: 0.6649999618530273 Average chunk time: 0.0)', 'Stream duration:', 0.6670000553131104) "<class 'lmql.runtime.bopenai.openai_api.OpenAIStreamError'>"
  warnings.warn("OpenAI: " + str(e) + ' "' + str(type(e)) + '"', category=OpenAIAPIWarning)
/home/pyodide/lmql/runtime/bopenai/batched_openai.py:669: OpenAIAPIWarning: Retrying 18 more times
  warnings.warn("Retrying {} more times".format(retries), category=OpenAIAPIWarning)
/home/pyodide/lmql/runtime/bopenai/batched_openai.py:681: OpenAIAPIWarning: OpenAI: ('This is a chat model and not supported in the v1/completions endpoint. Did you mean to use v1/chat/completions? (after receiving 0 chunks. Current chunk time: 0.5920000076293945 Average chunk time: 0.0)', 'Stream duration:', 0.5929999351501465) "<class 'lmql.runtime.bopenai.openai_api.OpenAIStreamError'>"
  warnings.warn("OpenAI: " + str(e) + ' "' + str(type(e)) + '"', category=OpenAIAPIWarning)
/home/pyodide/lmql/runtime/bopenai/batched_openai.py:669: OpenAIAPIWarning: Retrying 17 more times
  warnings.warn("Retrying {} more times".format(retries), category=OpenAIAPIWarning)
/home/pyodide/lmql/runtime/bopenai/batched_openai.py:681: OpenAIAPIWarning: OpenAI: ('This is a chat model and not supported in the v1/completions endpoint. Did you mean to use v1/chat/completions? (after receiving 0 chunks. Current chunk time: 0.28600025177001953 Average chunk time: 0.0)', 'Stream duration:', 0.2870001792907715) "<class 'lmql.runtime.bopenai.openai_api.OpenAIStreamError'>"
  warnings.warn("OpenAI: " + str(e) + ' "' + str(type(e)) + '"', category=OpenAIAPIWarning)
/home/pyodide/lmql/runtime/bopenai/batched_openai.py:669: OpenAIAPIWarning: Retrying 16 more times
  warnings.warn("Retrying {} more times".format(retries), category=OpenAIAPIWarning)
Initializing LMQL browser environment...
LMQL 0.999999 (dev) on Pyodide Python 3.10.2 (main, Jan 25 2023, 18:32:53) [Clang 16.0.0 (https://github.com/llvm/llvm-project dee009d3b5c71a340eb4ebb3cfb4

I tried this out in the LMQL playground.

Thanks for raising this. I identified the problem, and pushed a fix. I should be deployed to the web playground momentarily.


Find all duplicate subtrees in a binary tree.

• Brute Force -> Efficient Solution
• Step by step

Hey @aaryahjolia ,
I'm Darshan Jain, and I'm thrilled to be a part of this GSSOC'23. I bring along my expertise in Data Structures and Algorithms in C++. I'm passionate about open source projects and eager to contribute and collaborate with fellow enthusiasts.
Please assign this to me.


310 - Updated and added tests for afterOrAt according to matrix

Description
Updated existing tests and added new tests for afterOrAt according to the test matrix
Additional notes
All tests has been added and retested ignored tests related to #611  as well.
Issue
Resolves #310

Codecov Report

Merging #778 into master will increase coverage by 0.22%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #778      +/-   ##
============================================
+ Coverage     66.15%   66.37%   +0.22%
- Complexity     1757     1759       +2
============================================
  Files           279      279
  Lines          6154     6135      -19
  Branches        717      717
============================================
+ Hits           4071     4072       +1
+ Misses         1814     1792      -22
- Partials        269      271       +2

Impacted Files
Coverage Δ
Complexity Δ

...g/generator/restrictions/DataTypeRestrictions.java
62.85% <0%> (-2.86%)
13% <0%> (-1%)

...ogic/deg/generator/generation/VelocityMonitor.java
12.12% <0%> (+0.58%)
1% <0%> (-1%)
:arrow_down:

...nerator/walker/iterator/RouteDecisionIterator.java
100% <0%> (+4%)
10% <0%> (+1%)
:arrow_up:

...rator/restrictions/DateTimeRestrictionsMerger.java
78.78% <0%> (+9.09%)
17% <0%> (+3%)
:arrow_up:

Continue to review full report at Codecov.

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


RHOAIENG-371: Add pipeline support for git private repo using an authentication token

Description

This is blocked by https://github.com/opendatahub-io/ai-edge/pull/284
Users can now fetch from private repos, this pr adds that ability to the aiedge-e2e pipeline but once #284 is merged these changes will be moved there. And with that any docs that are needed to give users info on the new feature.
The e2e tests have beed updated to support this optional feature and have been passing when using the git_fetch test when cloning from my own private repo.
How Has This Been Tested?

Using e2e tests fill in token and username in config.json
All tests should pass

I have a private example repo if needed for testing confirmation
Merge criteria:

[x] The commits are squashed in a cohesive manner and have meaningful messages.
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has manually tested the changes and verified that the changes work

This now has conflicts after #284 was merged.

@MarianMacik fixed now

/hold
@jackdelahunt Putting this on-hold as some part of the git-auth implementation needs to be updates based on #290 and https://tekton.dev/docs/pipelines/git-resolver/

/retest

Hello @jackdelahunt, could you please rebase so that we have a clean CI and we can look at the changes in the context of the latest repo content?

@adelton done now

/unhold
/lgtm


[proposal] var env to stop the DB after init : INIT_DB_ONLY

In my case I have several tables to add.
Starting the docker image takes already a lot of time. When I want to test my project, I need to run it into a docker image to execute tests.
For this, at the moment I run the db then forces it to stop:
dockerfile
RUN cp container-entrypoint.sh container-entrypoint.sh.bak && sed -i 's/  echo "DATABASE IS READY TO USE!"/exit 0/g' container-entrypoint.sh
USER root
RUN chmod a+rwx /container-entrypoint-initdb.d/*
USER oracle
RUN ["container-entrypoint.sh"]
USER root
RUN rm container-entrypoint.sh && mv container-entrypoint.sh.bak container-entrypoint.sh
USER oracle

it would be good if the "echo database is ready to use" command would instead check if the INIT_DB_ONLY is set, and exit 0 in that case.
This way I could do
dockerfile
ENV INIT_DB_ONLY=true
RUN ["container-entrypoint.sh"]
ENV INIT_DB_ONLY=

instead.

Hi @glelouet,
Thank you for using these images!
I believe what you are looking for has already been provided as part of #119 with the --nowait option for container-entrypoint.sh.
Calling RUN ["container-entrypoint.sh", "--nowait"] will initialize the database and exit afterwards.

indeed, that sounds exactly what I should use

I tested and it seems to work :)

Great, glad to hear that! :)


Please document that /Tests parameter matches not just test name, but rather it's complete name (including the namespace and class name)

We spent half a day troubleshooting why /Tests:Blah executes all tests in the assembly, not just that one.
vstest.console.exe MyTests.dll /Tests:Blah
Microsoft (R) Test Execution Command Line Tool Version 17.3.0-preview-20220626-01 (x64)
Copyright (c) Microsoft Corporation.  All rights reserved.

Starting test discovery, please wait...
A total of 1 test files matched the specified pattern.
  Passed Abc [16 ms]   <-- unexpected
  Passed Blah [16 ms]
  Passed Xyz [< 1 ms] <-- unexpected

It turns out that matching is done over complete test name, including it's namespace.
Our tests were in something like
namespace Tests.Blah {
    [TestClass]
    public class MyTestClass
    {
        [TestMethod]
        public void MyTestMethod()
	}
	<...>
}

Since Blah is substring of full test name Tests.Blah.MyTestClass.MyTestMethod, this automatically opted in all the tests in the assembly.
This was hard to discover because output of vstest.console.exe does not show full namespace.

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

ID: e27af73d-196c-4653-aa07-acf7e8b425e3
Version Independent ID: b8957d80-ffa0-1c42-d3f0-6e4d00bf7a1e
Content: VSTest.Console.exe command-line options - Visual Studio (Windows)
Content Source: docs/test/vstest-console-options.md
Product: visual-studio-windows
Technology: vs-ide-test
GitHub Login: @Mikejo5000
Microsoft Alias: mikejo

@AbhitejJohn Abhitej, It seems to make sense to add a note to the effect that the command matches on the full test name. I'll open a PR, but let me know if you think otherwise (or if this is a bug).

@ayeltsov Thanks for the suggestion! I'll make an update that should go live by the end of the day.


onDuplicate not working

Using arangojs ^6.0.1 with latest node and arangodb.
When trying to save (updates to) an existing document via:
collection.save(doc, {onDuplicate: 'replace'})
it fails with
errorMessage: 'unique constraint violated - in index 0 of type primary over ["_key"]; conflicting key:
My expectation would be for arangodb to gracefully replace the document as per the documentation.

I don't think that collection.save has an onDuplicate parameter that would do that.
May I ask where you got the info from?
I can the task is to replace an existing document, there is also collection.replace, in case it should be just partially updated, there is collection.update.

Thanks for responding so fast - awesome support! I got it from here.
I essentially want an UPSERT operation from a node application. Will collection.replace do just that?

Ah, got it. I think that question referred to collection.import, which does a bulk insert/replacement of documents (in contrast to collection.save and collection.replace which are single-document operations).
As there is no first-class collection.upsert operation in arangojs yet, there are the following three options for performing it:

use a combination of fetching the document to check if it exists, then either insert it or update/replace it. This is the easiest option, however the sequence of operations will not be transaction-safe, meaning another parallel client operation could slip in between and insert the "same" document, which will cause a conflict. You can ignore this if you do not care about parallel writes of the "same" documents.
use an AQL UPSERT query. This will be transaction-safe but it will require you to run an AQL query in your client operation instead of something like collection.upsert. So it will be a bit more coding.
use a transaction from arangojs (https://www.npmjs.com/package/arangojs#databasetransaction) that can be executed server-side. That transaction will have proper semantics in case you need them.

I'm closing this issue as it looks like the question has been answered and there hasn't be a follow-up question in two months. If you have more questions about ArangoDB I recommend checking out our community slack at https://slack.arangodb.com/

@jsteemann Using the following query pattern, can many documents (10k+) be sent at once, or should one send batches (e.g. 1k at a time)?
FOR doc IN @docs
  UPSERT { _key: doc._key }
  INSERT doc
  UPDATE doc

{
  "docs": [
    {"_key": "upsert_test", "foo": "bar"},
    {"no": "key"}
  ]
}

BTW: This pattern works really nicely. If no key is provided for a document, a new document will be inserted with an auto-generated key. This is also the case if documents exists with key "null" or "undefined".


Hyperlink DOI to preferred resolver

Same as https://github.com/google/highwayhash/pull/81 ;-)

Will be included in the next release.

You're welcome :-)


Fix headers and format

Adds proper include guards for header files
Fixes ssize_t format in printing
Fixes dereferencing type-punned pointer warning

@hufrea what exactly you don't like in this PR? I am open to fix this too.

@dartvader316 sorry, I left a comment but didn't post (I thought you ignored it, sorry)


PREINSTALL.md file is not updated

Describe the bug
PREINSTALL.md file needs to be updated.
To Reproduce
Appliance role of Omnia has some Prerequisite, these prereqs are not updated in the PREINSTALL.md
Expected behavior
PREINSTALL.md should have updated prereqs.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: CentOS 7.9

Smartphone (please complete the following information):

NA

Additional context
NA

Updated PREINSTALL docs are uploaded as part of issue # 218.


Command not valid any longer

The section "Deploy" references the command: 'pac package deploy'. This command doesn't exist in the current version of the pac cli.

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

ID: 78dd2e38-9d0c-3bbf-0db3-138416d4063b
Version Independent ID: 51455bd4-3294-0ccd-2b1b-6d9857c0f50f
Content: Package Deployer tool - Power Platform
Content Source: power-platform/alm/package-deployer-tool.md
Service: power-platform
Sub-service: alm
GitHub Login: @marcelbf
Microsoft Alias: marcelbf

Thanks for your comment, @jcoxdco. This issue has been assigned to the doc author for review.


Bugfix/fix libp11 openssl again

Fixes oe-selftest -u updater.  This bug was caused by a change in upstream rocko


Broken tests: set_readable_signals does not accept Motor

Running the tests in dodal main against ophy_async main gives the following error:
  File "/scratch/ffv81422/hyperion/dodal/src/dodal/beamlines/i22.py", line 119, in slits_6
    return numbered_slits(
           ^^^^^^^^^^^^^^^
  File "/scratch/ffv81422/hyperion/dodal/src/dodal/utils.py", line 101, in wrapper
    return func(*args, **kwds)
           ^^^^^^^^^^^^^^^^^^^
  File "/scratch/ffv81422/hyperion/dodal/src/dodal/beamlines/_device_helpers.py", line 27, in numbered_slits
    return device_instantiation(
           ^^^^^^^^^^^^^^^^^^^^^
  File "/scratch/ffv81422/hyperion/dodal/src/dodal/utils.py", line 101, in wrapper
    return func(*args, **kwds)
           ^^^^^^^^^^^^^^^^^^^
  File "/scratch/ffv81422/hyperion/dodal/src/dodal/beamlines/beamline_utils.py", line 102, in device_instantiation
    device_instance = device_factory(
                      ^^^^^^^^^^^^^^^
  File "/scratch/ffv81422/hyperion/dodal/src/dodal/devices/slits.py", line 18, in __init__
    self.set_readable_signals(
  File "/scratch/ffv81422/hyperion/dodal/.venv/lib/python3.11/site-packages/ophyd_async/core/standard_readable.py", line 68, in set_readable_signals
    self.add_readables(read, wrapper=HintedSignal)
  File "/scratch/ffv81422/hyperion/dodal/.venv/lib/python3.11/site-packages/ophyd_async/core/standard_readable.py", line 209, in add_readables
    obj = wrapper(readable)
          ^^^^^^^^^^^^^^^^^
  File "/scratch/ffv81422/hyperion/dodal/.venv/lib/python3.11/site-packages/ophyd_async/core/standard_readable.py", line 238, in __init__
    assert isinstance(signal, SignalR), f"Expected signal, got {signal}"
AssertionError: Expected signal, got <ophyd_async.epics.motion.motor.Motor object at 0x7f423024a090>

This is happening because Slits is passing a Motor into set_readable_signals, which only accepts a Signal. @coretl do we expect Motor to be able to be passed in? In which case is this an ophyd-async error?
Acceptance Criteria

Tests are passing on main

After a long discussion in https://github.com/bluesky/ophyd-async/issues/192 we settled on https://github.com/bluesky/ophyd-async/issues/223, but implemented set_readable_signals on top of the new methods to ease adoption with a deprecation warning. However, it looks like the reimplemented set_readable_signals uses HintedSignal underneath which means we are now enforcing that it is a Signal as per the original type hints.
My preferred solution would be to switch from:
        self.x_gap = Motor(prefix + "X:SIZE")
        self.y_gap = Motor(prefix + "Y:SIZE")
        self.x_centre = Motor(prefix + "X:CENTRE")
        self.y_centre = Motor(prefix + "Y:CENTRE")

        # Type ignore pending: https://github.com/bluesky/ophyd-async/issues/192
        self.set_readable_signals(
            read=[
                self.x_gap,
                self.y_gap,
                self.x_centre,
                self.y_centre,
            ]  # type: ignore
        )

to
        with self.add_children_as_readables():
            self.x_gap = Motor(prefix + "X:SIZE")
            self.y_gap = Motor(prefix + "Y:SIZE")
            self.x_centre = Motor(prefix + "X:CENTRE")
            self.y_centre = Motor(prefix + "Y:CENTRE")

whenever you have non-signal readable children of a StandardReadable. Would that work for you?

Yep, that works, thanks @coretl


Modbus Gateway: Transaction ID Error!

samples/subsys/modbus/tcp_gateway/
fix the bug:
diff --git a/subsys/modbus/modbus_raw.c b/subsys/modbus/modbus_raw.c
index e4e330f5ad..e283dc7f8f 100644
--- a/subsys/modbus/modbus_raw.c
+++ b/subsys/modbus/modbus_raw.c
@@ -147,7 +147,7 @@ int modbus_raw_backend_txn(const int iface, struct modbus_adu *adu)
        err = modbus_tx_wait_rx_adu(ctx);

        if (err == 0) {
-               memcpy(adu, &ctx->rx_adu, sizeof(struct modbus_adu));
+               memcpy((uint8_t *)adu + 4, (uint8_t *)&ctx->rx_adu + 4, sizeof(struct modbus_adu) - 4);
        } else {
                modbus_set_exception(adu, MODBUS_EXC_GW_TARGET_FAILED_TO_RESP);
        }

Please send a PR for the fix so that it can be reviewed and merged properly.

Please send a PR for the fix so that it can be reviewed and merged properly.

It does not look like correct fix for me. And for what bug?

Please send a PR for the fix so that it can be reviewed and merged properly.

It does not look like correct fix to me. And for what bug?

I do not know, that is why I suggested that a PR is created so that the "fix" can be reviewed.


Can't deploy created by make dist-dev to AWS

$ make dist-dev
[...]
$ appsctl aws deploy app_google-drive_1.0.0.zip
2023-01-04T02:22:29.428+0100	INFO	upaws/provision_data.go:92	found manifest	{"file": "manifest.json"}
2023-01-04T02:22:29.428+0100	INFO	upaws/provision_data.go:120	found static asset	{"file": "static/google.png"}
2023-01-04T02:22:29.428+0100	INFO	upaws/provision_data.go:120	found static asset	{"file": "static/drive.png"}
2023-01-04T02:22:29.428+0100	INFO	upaws/provision_data.go:104	found lambda function bundle	{"file": "app_google-drive_1.0.0.zip"}
Error: can't get deploy data from file app_google-drive_1.0.0.zip: deploy data is not valid: 4 errors occurred:
	* different number of functions in the manifest and in the bundle
	* function {/ bundle bundle/index.handler nodejs16.x} was not found in the bundle
	* mismatched handler
	* mismatched runtime

Works with the latest code in https://github.com/mattermost/mattermost-app-google-drive/pull/2. Closing.


[ALLUXIO-2739] Fix "runMiniBentchmark" command to respect master addr set in Configuration

https://alluxio.atlassian.net/browse/ALLUXIO-2739

PTAL

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/14304/
Test PASSed.

Thanks for the fix. I left two minor comments.
Overall it looks good to me.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/14319/
Test PASSed.


I find anyone who find this problem :unauthorized access to "libmedia.so"

When I start, pause, and resume this sample app, this popup message is represented:
Detected Problem with app native libraries
(Please consult log for details):
libcallrecfix.so: unauthorized access to "libmedia.so"
And When I start the record and finish the phone call, the file is saved with .WAV file, and it doesn't have any sounds
What's the problem?

same for me with Android O

solved.

Could you please share your solution ?

In my case, just build app in release version.


Use NuGet packages for Gtk/Cairo stuff.

So as not to need a separate install of GtkSharp. This way, we can move to a single solution for both Window and Gtk.

Seems that the NuGet packages don't, and can't work - for now at least.


Fix ECMA-402 declarations (issue #10618)

Fixes #10618

Make String.prototype.localeCompare's locales parameter optional, so undefined is allowed.
Declare the locales parameter as a string | string[] union instead of using overloads. Having separate overloads for string and string[] unnecessarily prevents passing a string | string[]. (These overloads predate the introduction of union types.)

@jeffreymorlan, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, MSBOT;

thanks!


Remove init() from KemInterface.

Codecov Report

Merging #244 (b3a1254) into main (109a852) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #244      +/-   ##
==========================================
- Coverage   97.01%   97.00%   -0.02%
==========================================
  Files          34       34
  Lines        2514     2505       -9
  Branches      225      225
==========================================
- Hits         2439     2430       -9
  Misses         74       74
  Partials        1        1

Flag
Coverage Δ

unittests
97.00% <ø> (-0.02%)
:arrow_down:

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

Files Changed
Coverage Δ

src/cipherSuiteNative.ts
100.00% <ø> (ø)

src/kems/dhkem.ts
100.00% <ø> (ø)


Crash caused by adding a field

Steps to Reproduce
My application added a field to the model and reported many crash errors that were severe and difficult to eliminate. The following is an error report
If you add an age field, after adding it, I have regenerated the. g.dart file

OS Version: iOS 16.6 (20G75)
Report Version: 104

Exception Type: EXC_BAD_ACCESS (SIGBUS)
Exception Codes: BUS_NOOP
Crashed Thread: 4

Application Specific Information:
Exception 1, Code 0, Subcode 8 >
Attempted to dereference null pointer.

Thread 14 name: isarworker
0   libsystem_kernel.dylib          0x3c263dca8         __psynch_mutexwait
1   libsystem_pthread.dylib         0x404334114         <redacted>
2   libsystem_pthread.dylib         0x40433b318         <redacted>
3   isar_flutter_libs               0x105c5f118         <unknown> + 324
4   isar_flutter_libs               0x105c5ed50         <unknown> + 196
5   isar_flutter_libs               0x105c5e7d0         <unknown> + 36
6   libsystem_pthread.dylib         0x4043336b8         _pthread_start

Thread 15 name: isarworker
0   libsystem_kernel.dylib          0x3c263d578         __psynch_cvwait
1   libsystem_pthread.dylib         0x40433a078         <redacted>
2   isar_flutter_libs               0x105c58dd8         <unknown> + 152
3   isar_flutter_libs               0x105c5f698         <unknown> + 1732
4   isar_flutter_libs               0x105c5ed50         <unknown> + 196
5   isar_flutter_libs               0x105c5e7d0         <unknown> + 36
6   libsystem_pthread.dylib         0x4043336b8         _pthread_start

Thread 16 name: isarworker
0   libsystem_kernel.dylib          0x3c263dca8         __psynch_mutexwait
1   libsystem_pthread.dylib         0x404334114         <redacted>
2   libsystem_pthread.dylib         0x40433b318         <redacted>
3   isar_flutter_libs               0x105c5f118         <unknown> + 324
4   isar_flutter_libs               0x105c5ed50         <unknown> + 196
5   isar_flutter_libs               0x105c5e7d0         <unknown> + 36
6   libsystem_pthread.dylib         0x4043336b8         _pthread_start

Thread 17 name: isarworker
0   libsystem_kernel.dylib          0x3c263dca8         __psynch_mutexwait
1   libsystem_pthread.dylib         0x404334114         <redacted>
2   libsystem_pthread.dylib         0x40433b318         <redacted>
3   isar_flutter_libs               0x105c5f118         <unknown> + 324
4   isar_flutter_libs               0x105c5ed50         <unknown> + 196
5   isar_flutter_libs               0x105c5e7d0         <unknown> + 36
6   libsystem_pthread.dylib         0x4043336b8         _pthread_start

Thread 18 name: isarworker
0   libsystem_kernel.dylib          0x3c263dca8         __psynch_mutexwait
1   libsystem_pthread.dylib         0x404334114         <redacted>
2   libsystem_pthread.dylib         0x40433b318         <redacted>
3   isar_flutter_libs               0x105c5f118         <unknown> + 324
4   isar_flutter_libs               0x105c5ed50         <unknown> + 196
5   isar_flutter_libs               0x105c5e7d0         <unknown> + 36
6   libsystem_pthread.dylib         0x4043336b8         _pthread_start

OS Version: iOS 16.0 (20A362)
Report Version: 104

Exception Type: EXC_BAD_ACCESS (SIGBUS)
Exception Codes: BUS_NOOP at 0x0000000000001844
Crashed Thread: 2

Application Specific Information:
Exception 1, Code 6212, Subcode 8 >
Attempted to dereference garbage pointer 0x1844.

Thread 13 name: isarworker
0   libsystem_kernel.dylib          0x392ae3b48         __psynch_mutexwait
1   libsystem_pthread.dylib         0x3b2f5e150         <redacted>
2   libsystem_pthread.dylib         0x3b2f65310         <redacted>
3   isar_flutter_libs               0x10752b118         <unknown> + 324
4   isar_flutter_libs               0x10752ad50         <unknown> + 196
5   isar_flutter_libs               0x10752a7d0         <unknown> + 36
6   libsystem_pthread.dylib         0x3b2f5d6cc         _pthread_start

Thread 14 name: isarworker
0   libsystem_kernel.dylib          0x392ae3b48         __psynch_mutexwait
1   libsystem_pthread.dylib         0x3b2f5e150         <redacted>
2   libsystem_pthread.dylib         0x3b2f65310         <redacted>
3   isar_flutter_libs               0x10752b118         <unknown> + 324
4   isar_flutter_libs               0x10752ad50         <unknown> + 196
5   isar_flutter_libs               0x10752a7d0         <unknown> + 36
6   libsystem_pthread.dylib         0x3b2f5d6cc         _pthread_start

Thread 15 name: isarworker
0   libsystem_kernel.dylib          0x392ae341c         __psynch_cvwait
1   libsystem_pthread.dylib         0x3b2f6406c         <redacted>
2   isar_flutter_libs               0x107524dd8         <unknown> + 152
3   isar_flutter_libs               0x10752b698         <unknown> + 1732
4   isar_flutter_libs               0x10752ad50         <unknown> + 196
5   isar_flutter_libs               0x10752a7d0         <unknown> + 36
6   libsystem_pthread.dylib         0x3b2f5d6cc         _pthread_start

Thread 16 name: isarworker
0   libsystem_kernel.dylib          0x392ae3b48         __psynch_mutexwait
1   libsystem_pthread.dylib         0x3b2f5e150         <redacted>
2   libsystem_pthread.dylib         0x3b2f65310         <redacted>
3   isar_flutter_libs               0x10752b118         <unknown> + 324
4   isar_flutter_libs               0x10752ad50         <unknown> + 196
5   isar_flutter_libs               0x10752a7d0         <unknown> + 36
6   libsystem_pthread.dylib         0x3b2f5d6cc         _pthread_start

Thread 17 name: isarworker
0   libsystem_kernel.dylib          0x392ae3b48         __psynch_mutexwait
1   libsystem_pthread.dylib         0x3b2f5e150         <redacted>
2   libsystem_pthread.dylib         0x3b2f65310         <redacted>
3   isar_flutter_libs               0x10752b118         <unknown> + 324
4   isar_flutter_libs               0x10752ad50         <unknown> + 196
5   isar_flutter_libs               0x10752a7d0         <unknown> + 36
6   libsystem_pthread.dylib         0x3b2f5d6cc         _pthread_start

Thread 18 name: isarworker
0   libsystem_kernel.dylib          0x392ae3b48         __psynch_mutexwait
1   libsystem_pthread.dylib         0x3b2f5e150         <redacted>
2   libsystem_pthread.dylib         0x3b2f65310         <redacted>
3   isar_flutter_libs               0x10752b118         <unknown> + 324
4   isar_flutter_libs               0x10752ad50         <unknown> + 196
5   isar_flutter_libs               0x10752a7d0         <unknown> + 36
6   libsystem_pthread.dylib         0x3b2f5d6cc         _pthread_start

Details

Platform: iPhone
Flutter version: [3.7.11]
Isar version: [3.0.2]

It needs to be uninstalled and reinstalled to recover without crashing. Apple has appeared more frequently

Please update your Isar version to resolve this issue

Please update your Isar version to resolve this issue

Did the latest version fix this issue?

Please update your Isar version to resolve this issue

I need to confirm that upgrading isar works, because this problem cannot be re-lined


Cow model

Need a sample cow model asset to embody mooishness.

Moo, I'll see what can be done about making the viewer more mooish.

@karolinepauls Here's a "cow" for you to embody mooishness.


Method not found: 'ButtonTheme.bar'

When I try to run my app in chrome, I keep getting this error:
../../../../AppData/Local/Pub/Cache/hosted/pub.dartlang.org/date_range_picker-1.0.6/lib/date_range_picker.dart:1152:44: Error: Method not found: 'ButtonTheme.bar'.     final Widget actions = new ButtonTheme.bar(
I just updated flutter to the last version of and as posted in this post in StackOverflow, ButtonTheme.bar is deprecated. This results in not being able to run the app in web. I don't use "directly" ButtonTheme, but the package flutter_form_builder does. Someone already posted the issue in flutter_form_builder.
Maybe a solution would be to modify date_range_picker yourself to replace the ButtonTheme with ButtonBarTheme...

This seems to be a blocker on Flutter 2.0.0 stable.

@alexmarkley  I am having same issue, how do i revert back or is there some otherway. Please help. I have deadline ahead of me

Any one with a fix?

For temporary fix .fromButtonThemeData instead of  .bar

@Shiba-Kar downgrade to previous version of flutter, using "flutter downgrade"

@faisalmushtaq007
Until the owner of the package gives a fix, here is a temporary fix which is much easier.
In your pubspec.yaml remove the dependency of date_range_picker and replace it with below dependency
dependency_overrides:
  date_range_picker:
    git:
      url: git://github.com/justkawal/date_range_picker.git
      ref: master

~ Best
~ justkawal
~ Long Live 🔥pen Source

This lib is too awesome compared to Flutter's own DatePicker...
I've forked & upgraded this to null safe here: https://github.com/tcd93/date_range_picker


Create Bucket elimination algorithm

Bucket elimination is a fast algorithm that works well for networks with small "tree width".
Positive examples include: TTN, quantum circuits, MPS, and MERA.
Negative examples include: SAT TensorNetwork.
Seeing how most of our work involves the former, we should implement this algorithm asap. @viathor has been working on a first version.

Note that BE contractor exploits sparsity in the tensors by reducing the number of indices to range over in cases where terms corresponding to most index combinations are zero. This arises very often in quantum circuits where a lot of tensors are diagonal.
Treewidth sets the lower bound for contraction complexity.
This should work fairly well on quantum circuits and SAT tensor network. For the other types, a lot depends on the tensors, but note that performing SVD on an edge does introduce a diagonal tensor that BE contractor can exploit.


Best way to authenticate requests

I am trying to figure out the best way to authenticate a request before my schema executes the query/mutation. I was thinking I could use the @app.resolver('url') tag and create a function that would authenticate the request and then pass it to the graphqlview. Something like this:
@app.route('/graphql/')
def graphql():
    print request.headers #AUTHENTICATE HERE
    print request.data
    return GraphQLView.as_view('/graphql/', schema=schema, graphiql=True)

But this doesn't work, can someone either show me what I am doing wrong here or give me a different way to authenticate the request? Thanks!

I had a similar problem, but I basically implemented a workaround such that a user could access the graphqlview only on the testing/development environment (localhost). Depends on what your goal is, I only had the graphqlview to help me model and test graphql queries, and didn't actually want it available in the production environment. Here's the code if you're interested:
   isLocal = 'FLASK_DEBUG' in os.environ and os.environ['FLASK_DEBUG'] == '1'
   app.add_url_rule('/graphql', view_func=GraphQLView.as_view('graphql', schema=schema, context={ 'client': MONGO_CLIENT }, graphiql= True if isLocal else False))

@kmakihara Did you end up writing individual graphql routes that handle authentication? How are you limiting the amount of data an authenticated user can potentially receive without limiting the usefulness of graphql?

It's been a while that issue is open, but for general purpose, I ended up using route decorators in my view function like this :
from flask_jwt_extended import jwt_required
...

def graphql_view():
    view = GraphQLView.as_view('graphql', schema=schema, context={'session': db.session},
                               graphiql=True)
    return jwt_required(view)

app.add_url_rule(
    '/graphql',
    view_func=graphql_view()
)

Here I use a JWT authentication with but I could use flask-login or any authentication method:
from flask_login import login_required
...

def graphql_view():
    view = GraphQLView.as_view('graphql', schema=schema, context={'session': db.session},
                               graphiql=True)
    return login_required(view)
...

Here is maybe a more intuitive way for those who don't use jwt:
`
def auth_required(fn):
@wraps(fn)
def wrapper(*args, **kwargs):
session = request.headers.get(AUTH_HEADER, '')
# Do some authentication here maybe...
return fn(*args, **kwargs)
return wrapper
def graphql_view():
    view = GraphQLView.as_view(
        'graphql',
        schema=schema,
        graphiql=True,
        context={
            'session': DBSession,
        }
    )
    return auth_required(view)

app = Flask(__name__)
app.debug = True
app.add_url_rule(
    '/graphql',
    view_func=graphql_view()
)

`

I am using flask + jwt.
For queries I just use Viewer, as a top level object:
class Query(graphene.ObjectType):
    ....
    viewer = graphene.Field(Viewer)

    @staticmethod
    def resolve_viewer(_root, info):
        return Viewer.get_user_by_token(info.context.headers.get('Authorization'))

For mutations I have to create abstract mutation class and extend it with all my mutation except login mutation:
class AuthorizedMutation(relay.ClientIDMutation):
    class Meta:
        abstract = True

    @classmethod
    @abstractmethod
    def mutate_authorized(cls, root, info, **kwargs):
        pass

    @classmethod
    def mutate_and_get_payload(cls, root, info, **kwargs):
        _ = auth_service.authorize_token(info.context.headers.get('Authorization'))
        return cls.mutate_authorized(root, info, **kwargs)

All credentials and tokes related work is handeled by auth_service. Viewer.get_user_by_token also calls it.
Hope it will be helpful.

Hi, I'm the maintainer of Flask-GraphQL-Auth. Inspired by Flask-JWT-Extended, There is a problem with error-handling but it works pretty well.  How about try this?
You can use Flask-GraphQL-Auth like you used Flask-JWT-Extended.
here are some examples.
class Query(graphene.ObjectType):
    protected = graphene.String(message=graphene.String(),
                                token=graphene.String())

    @jwt_required
    def resolve_protected(self, info, message):
        return str(get_raw_jwt())

you can find more on github and docs
GitHub: https://github.com/callsign-viper/Flask-GraphQL-Auth
Docs: https://flask-graphql-auth.readthedocs.io/en/latest/


[20051] Fix redefinition of wrap___eq_

eProsima/Fast-DDS#4045 introduced new == operators for ReturnCode_t, removing the existing template ones. This has caused a redefinition of _wrap___eq__ on the python bindings generated code. This PR fixes that issue by ignoring the new operators, thus keeping the same API in the bindings.
Fast DDS docs will not build until this PR is merged.

@mergifyio backport 1.1.x 1.2.x 1.3.x


Gradle 3.3 -> 3.4 Annotation Processor not called in JavaCompile Task

If I use the JavaCompile Task with compilerArgs Options, then the Annotation Processor "SQLProcessor" is not called anymore. With gradle 3.0,3.1,3.2, 3.3 it is working. With gradle 3.4 it seems to be broken.
The Java source files and the annotation processor are in the same project.
Expected Behavior
Same result as with gradle 3.3.
Current Behavior
Annotation Processor is not called anymore.
Context
Steps to Reproduce (for bugs)
This task does not call the Annotation Processor anymore.
task generateSQLConst (type: JavaCompile) {
   description = "Creates SQL Constants"
   destinationDir = file("$buildDir/classes/sql")
   source = file("$projectDir/src/de/")
   includes = ['**/*.java']
   classpath = sourceSets.main.output + configurations.compile + configurations.provided
   options.compilerArgs = ['-processor', 'de.util.annotation.SQLProcessor']
}

Your Environment
Java 7, Gradle 3.4

Could you please push a reproducible example to GitHub? That would make it easier for us to fix the problem.

I think I know what the issues is though: Your annotation processor is missing the processor configuration file (META-INF/services/javax.annotation.processing.Processor, see javac documentation). Gradle 3.4 scans for these configuration files and passes the proc:none flag if it doesn't find any.
Apparently javac allows you to use a processor with the explicit -processor flag even if it does not have a valid processor configuration file. So I guess Gradle should be lenient in this case as well.
As a workaround, add a processor configuration file for your processor. That way you don't have to pass -processor anymore, making your task setup simpler.

@melix This should be a simple fix we can do between branching and 3.5-RC-1. We need to check for an explicit -processor flag here.

Thanks for  your fast reply.
Your solution proposal worked for us. We added META-INF/services/javax.annotation.processing.Processor and removed the "-processor" flag.

@oehme can you take a quick look at the fix above and close this issue if you are happy with it?

LGTM


Running server in production mode, static resources 404 error

Using grunt serve is OK, everything is normal;  that's in development mode.
However, using grunt serve:dist, or go to dist folder and run node server.js. bower javascript components and fonts are not served, which cause 404 errors when visiting the site .
GET /bower_components/jquery/jquery.js 404 7ms - 138b
GET /fonts/fontawesome-webfont.woff?v=4.0.3 404 4ms - 138b

Add following two lines in server.js also not helping
app.use(express.static(__dirname + '/client'));
app.use(express.static(__dirname + '/client/styles'));

I am using Backbone, requirejs and express/mongodb.
This could be a bug, so I post here rather than in Stackoverflow.
Thanks!

Could you post your .yo-rc.json file? This will help me debug the issue you are having.

It's like this.

Hey @viewplatgh,
It looks like you are using RequireJS's configuration for third-party/global libs (jquery, jstree, etc). This is the issue since you should be using the usemin blocks (<!-- build:js -->) within your index.html file to load your third-party scripts.
 You can remedy this by just using the global objects that the third-party libs create (ex. jquery attaches to window.$ and window.jQuery)
Hopefully that makes sense. This issue you are having is technically not a bug, just a process issue. If you follow the mantra that all global/third-party scripts are loaded within index.html and you just use their window.* object references in your scripts, you should be able to get up and running in both grunt serve and grunt serve:dist


Add Win7 and Win8 agents to pull requests

As there are many OS specific nuances with IISIntegration, we should enable win7 and win8 jenkins agents for PR builds.

Horp.


marquee

add a marquee with some information just above the heading

I've done so along with a few other changes from before. Please view all my commits and merge my pull request. I've also resolved issue #16 in the same and the changes have been approved.

add a marquee with some information just above the heading

May i work on this ? i.e. issue #18


Update to PyO3 0.22

Due to the changes in https://github.com/PyO3/rust-numpy/pull/442, this crate does not build python packages for 32-bit Windows anymore.


Seems Slow download

Have review previous comments about slow speeds.
I am on a MacOs with latest updates applied, installed youtube-dl via brew.
Seems very slow on downloading with a 300Mb connections, is there throttling happening or something similar.
Says its going to be over 3hours to download Just under 1Gb

Duplicate of #29326 (etc).
You need to install the program from the git master branch, as discussed in the links from the above.


Introduce a request counter to the auditor

What type of PR is this?
/kind feature

What this PR does / why we need it:
This change creates a simple request counter, able to concurrently keep track of all HTTP requests  made to GCR. The counter is only enabled when the user passes the --verbose flag. In the future, this counter will be logged to gain visibility in the network performance of the auditor.
Supersedes #356
Which issue(s) this PR fixes:
Partially satisfies #358

Special notes for your reviewer:
The auditor does not yet call Increment() to add to the counter. This would perpetually increase the value of the counter if enabled. Once the concurrent logger is implemented (part of #358), which resets the counter every 10min, will the auditor call Increment().
Less importantly, this package is made specifically for the auditor, it can just as easily be used for other commands, like cip run --snapshot.
Does this PR introduce a user-facing change?
NONE

Triggering request counter for the auditor when --verbose flag is present.

cc: @listx @amwat @justaugustus @kubernetes-sigs/release-engineering

Thanks!
/lgtm
/approve


Caching

I think the library would benefit from having a caching strategy for downloaded or processed files. One option is to download the files outside of pandas and use an http cache. This tends to be temporary, no more than 7 days. Another option is to give the user the option to read from and save to a cache, which could just involve reading/writing parquet files from the existing data directory, which would be more permanent, or to the system tmp directory, which is more transient. Let me know if either option is of interest.

That sounds like a great idea, the second option that would be more permanent sounds like the preferable option to me but let me know if you think we'd be better off going a different direction.

OK, I'll put together a plan and get back to you.

I'd recommend using the user's system's cache directory for doing so. appdirs is a simple module for determining that location. Happy to help if you'd like.

Thanks for pointing us to appdirs - I hadn't seen library that before and it is very handy. In terms of storage strategy, I was looking at a partitioned parquet file by year for the play-by-play. My instinct is that the weekly / seasonal player and team stats and the other datafiles are not large enough to worry about partitioning.

Yeah that's the only one worth the effort

Had a chance to take a run at this this morning - newest release includes a cache_pbp() function for local caching. I'm going to go ahead and close this issue and if anyone has recommendations for further improvements to that function we can open up another one.


support of git/github links automatic linking

in the context of gitbucket/gitbucket#1323 and extending commonmark-java, I'd like to add the possibility to discover links from:

git SHA1 references
issues & PR ids

The "autolinked references" I'd like to detect are those described in github documentation.
I am currently implementing this, but it requires some internal changes in the project because several scanners will be able to be fired for the same characterset (for example for '@' or [a-zA-Z]). Thus I'd like to know if you would accept such changes. The changes I have in mind:

registration of Scanners would be ordered (to establish some precedence)
each scanner would be responsible of knowing on which characters it need to be triggered (inversion of responsibility between Scanner & LinkExtractor)
for each scan (ie LinkIterator#setNext()), first Scanner answering a Link would win

Hey! Interesting.
How would you detect commit hashes without having to check on almost every single character? And then if you have multiple scanners for the same character, I wonder how you can make that efficient enough. It seems like this might be something that a regex might be better for, as the patterns look fairly regular. Have you experimented with that?
(I realize this might sound negative, but I'm just a bit skeptical.)

@robinst I do not take it 'negative' ;-) I was expecting this because what I asked/proposed is a bit also against the way (or at least disruptive) your project is currently working. Of course a regexp would play nicely but not less than for existing recognitions (www scanner, URL scanner).


Update metadata.csv

https://datadog.zendesk.com/agent/tickets/1155395
What does this PR do?

The customer called us there was a typo in the document.
Motivation

To collect a typo
Additional Notes

https://datadog.zendesk.com/agent/tickets/1155395
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached
[ ] If the PR doesn't need to be tested during QA, please add a qa/skip-qa label.

Merging as the CI issue is not related to this change.
The update will be visible a bit later today


Use a RefCell to provide interior mutability for VolumeManager.

VolumeManager now contains a RefCell, so all the methods can be &self. This also means you can have multiple smart handles usable at the same time.
Note that VolumeManager is !Sync, so this is just about having multiple File objects active at one time.
Closes #79 (finally!)

I rebased.

Rust is brilliant. I had very little trouble chopping up this codebase and hacking on that shell example to use the auto-closing Directory objects. If it compiles, it has basically worked.

Rebased again 😢
Make small PRs, they said. It's easier to review, they said.

Make small PRs, they said. It's easier to review, they said.

Don't worry. It is easier to review!

I wonder if I should turn off the thing that dismisses reviews as stale when a commit is added.

OK, all rebased and notes added about double locking. I think this monster is good to go.


Distributed DataParallel support

Hi, nice project! I wonder when will LibMTL support Distributed DataParallel? It helps a lot!

Since many methods in LibMTL manipulate the gradients, DDP implementation will be complicated. We will work on it but it won't be supported soon.

Closed as no further updates.


HIP 82: Add Helium Mobile as a Service Provider to the Helium Mobile subDAO

HIP 82: Add Helium Mobile as a Service Provider to the Helium Mobile subDAO

Author(s): @zer0tweets, @meowshka (Nova Labs, Inc.)
Start Date: 2023-03-15
Category: economic, technical
Original HIP PR: #581
Tracking Issue: #628

Summary
This HIP proposes to introduce Helium Mobile as a Service Provider (as defined in HIP53) on the Mobile Network. The main goal is to bring usage to the Network and pave the way for other Service Providers to join. The HIP describes integration with Helium Mobile Network, anti-gaming mechanism, and overall impact on the Network.
Motivation
The Community has gone a long way in building Helium Mobile Network and now is the perfect time to bring usage and utility to it.
As a Service Provider member of the Helium Mobile subDAO, Helium Mobile will actively offload its subscriber traffic into the Helium Mobile Network.  The launch of the Helium Mobile carrier will help pave the path for other Service Providers coming to the Network by ironing out any undiscovered issues and creating a smooth onboarding process.  In addition, the increased traffic on the network as a result of the launch will have a commensurate increase in HNT burn and data rewards for Hotspot operators.
Rendered View
https://github.com/helium/HIP/blob/main/0082-helium-mobile-service-provider.md

Note: #636 was merged into this file and adds info about one year grace period to allow for zero-rated traffic

HIP 82 was approved by the community on 16th June 2023 with a passing rate of 99.9%
https://realms.heliumvote.com/dao/mobile/proposal/3u2DUcZ2GMRviU5Ycy827urBtovutUiGrt3jqJHQek3w


Setting margin to anything but [x,0] radically increases height of container.

I am trying to get fixed size of grid items by fixing the width and height of the parent container.
Everything works, but height. For example, this is my grid definition:
layout: {
        0: {x: 0, y: 5, w: 50, h: 70, i: "0"},
        1: {x: 50, y: 0, w: 21, h: 30, i: "1"},
        2: {x: 71, y: 9, w: 15, h: 21, i: "2"},
        3: {x: 50, y: 30, w: 30, h: 40, i: "3"},
    },
    cols: 86,
    rowHeight: 5,

This should roughly equate to a height of 375 px ((layout[0].y + layout[0].h) * rowHeight). And, it works nicely when margin={[0,0]} or margin={[50,0]}.
However, if margin is anything but {[x,0]}, height gets completely out of hand (e.g. margin={[0,5]} messes with the height of all grid items (makes them much larger)) . Unfortunately, I need the margins.
Any ideas?

One more bug - x margin eats into width of elements. Is that normal?

height:
h === Infinity
? h
: Math.round(rowHeight * h + Math.max(0, h - 1) * margin[1])
Above issue happening because of muliptlication of margin size into height grid units. and adding them into height.
what is the purpose of this?


samples: add ADXL362 sensor sample application

Add a sample application that demonstrates how to use the ADXL362 with
data ready and threshold triggers.
This depends on #15913 to work.

Tested on cc1352r1_launchxl board with SPI support from #15741 and a TI BOOSTXL-ULPSENSE shield not upstream yet.

rebased on master to re-kick CI.


Updates the README to add links to the examples folder and remove the…

… discussion of package name changes
Fixes #194

Coverage remained the same at 76.125% when pulling c7e8af78c126fbc1c489183de37e47a0be648f73 on examples into 372da029d8a7c57b402527235fc5d3996f5f509f on master.


Pass valid context to CompleteAuthToken function in Negotiate Security Package.

Sponsored by: WHEEL Systems

@freerdp-bot test.


Future Maintenance and Updates for flutter-tflite

Thank you for your work on the flutter-tflite package. However, I’m concerned about its future maintenance:
Last Update: The package hasn't seen updates since December 2023.
Potential Blockers: Without ongoing maintenance, there's a risk that this package might become a blocker for future Flutter or other package upgrades.
Pub.dev Issues: The package currently fails static analysis on Pub.dev, and dependencies are outdated. Keeping these up-to-date is essential for continued compatibility.
We understand that you may be busy with other projects, just as we are, but having a clear status on this package is crucial for us. This is the only solid package for integrating TensorFlow Lite with Flutter, and knowing its future will greatly impact our planning.
Can you please provide a clear answer on whether this package will continue to be maintained and updated? Having this information will help the community make informed decisions moving forward.
@PaulTR and other maintainers, please try to find some time to provide an answer on this.

Hi @PaulTR, could you please merge this branch? It has been tested and is stable with the latest Flutter SDK. Thanks!

Alright 0.11.0 is published.
As far as future maintenance plans - this is very much a community driven project at this point. If folks using it would like to also update it, that'd be awesome (which is what happened for this most recent fix, adding various OS fixes, and updating it after that 3 year pause when this was on a third-party repo). They'll have more hands-on experience and know what issues are there than I will for now (unfortunately). I'm basically jumping in to run terminal commands and publish when I can (compared to the previous setup of nothing happening for ~3 years).

Thanks @PaulTR !!!!


Binder Error: No function matches the given name and argument types '@(DATE)'. You might need to add explicit type casts.

I'm getting the following exception when trying to use Dapper and query using a DateTime parameter:
Unhandled exception. DuckDB.NET.Data.DuckDBException (0x00000001): Binder Error: No function matches the given name and argument types '@(DATE)'. You might need to add explicit type casts.
        Candidate functions:
        @(TINYINT) -> TINYINT
        @(SMALLINT) -> SMALLINT
        @(INTEGER) -> INTEGER
        @(BIGINT) -> BIGINT
        @(HUGEINT) -> HUGEINT
        @(FLOAT) -> FLOAT
        @(DOUBLE) -> DOUBLE
        @(DECIMAL) -> DECIMAL
        @(UTINYINT) -> UTINYINT
        @(USMALLINT) -> USMALLINT
        @(UINTEGER) -> UINTEGER
        @(UBIGINT) -> UBIGINT
        @(UHUGEINT) -> UHUGEINT

LINE 1: ...T * FROM items WHERE TransactionDate > @TransactionDate
                                                  ^
   at DuckDB.NET.Data.PreparedStatement.PrepareMultiple(DuckDBNativeConnection connection, String query, DuckDBParameterCollection parameters)+MoveNext()
   at DuckDB.NET.Data.DuckDBDataReader.InitNextReader()
   at DuckDB.NET.Data.DuckDBDataReader..ctor(DuckDbCommand command, IEnumerable`1 queryResults, CommandBehavior behavior)
   at DuckDB.NET.Data.DuckDbCommand.ExecuteDbDataReader(CommandBehavior behavior)
   at System.Data.Common.DbCommand.ExecuteDbDataReaderAsync(CommandBehavior behavior, CancellationToken cancellationToken)

Here is code to reproduce:
using Dapper;
using DuckDB.NET.Data;

await File.WriteAllTextAsync("./items.csv", """
    ItemId,TransactionDate,Quantity
    1001,2024-04-11,1
    1002,2024-04-11,1
    1003,2024-04-11,1
    1003,2024-04-10,1
    """);

using var connection = new DuckDBConnection("DataSource=:memory:");
connection.Open();

await connection.ExecuteAsync("CREATE TABLE items as SELECT * FROM read_csv('./items.csv')");

var items = await connection.QueryAsync<Item>(
    sql: "SELECT * FROM items WHERE TransactionDate > @TransactionDate",
    param: new { TransactionDate = new DateTime(2024, 04, 10) });

Console.WriteLine($"Loaded {items.AsList().Count} items");

internal class Item
{
    public long ItemId { get; set; }
    public DateTime TransactionDate { get; set; }
    public int Quantity { get; set; }
}

Have you checked docs for parameterized statements?

No. But I’ll test an example using that approach to rule out Dapper.

Ok - so my problem was using a named parameter with @ instead of $. Thanks for turning me on to that.
May consider supporting parameters as @ as since that is more common, I think. (At least my experience).
Thanks for your work on this. It's awesome!
Final working snippet:
using Dapper;
using DuckDB.NET.Data;

await File.WriteAllTextAsync("./items.csv", """
    ItemId,TransactionDate,Quantity
    1001,2024-04-11T00:00:00,1
    1002,2024-04-11T00:00:00,1
    1003,2024-04-11T00:00:00,1
    1003,2024-04-10T00:00:00,1
    """);

using var connection = new DuckDBConnection("DataSource=:memory:");
connection.Open();

await connection.ExecuteAsync("CREATE TABLE items as SELECT * FROM read_csv('./items.csv')");

var items = await connection.QueryAsync<Item>(
    sql: "SELECT * FROM items WHERE TransactionDate > $TransactionDate",
    param: new { TransactionDate = new DateTime(2024, 04, 10) });

Console.WriteLine($"Loaded {items.AsList().Count} items");

internal class Item
{
    public long ItemId { get; set; }
    public DateTime TransactionDate { get; set; }
    public int Quantity { get; set; }
}


add reboot scripts for all OSs and also setup instructions

Can you squash some of these commits first?


connect planning and vision nodes

Closes #50

I would maybe think about using a header file or a class definition stub as the layout has become quite confusing especially with the private block at the end.
Otherwise this looks good to me, but I don't have deep knowledge here

Also closes https://github.com/UHHRobotics22-23/marimbabot/issues/50

Closes https://github.com/UHHRobotics22-23/marimbabot/issues/58 as well

As it is now, the ActionServer receives a LilyPond String as an Action Goal and forwards it to "lilypond_to_hitpoint.cpp".
@berkgungor @Flova  @scaredycode
We should get together and

talk about client side (ActionClient), that should be straight forward
talk about the robot side, i.e. the data representation that is needed/suitable in order to execute a music sequence including dynamics, note values and bpm maybe. This assumes that Lilypond Parsing (to a MIDI representation or whatever) happens inside the ActionServer or somewhere later. :)

TODO adjust lilypond_to_hitpoint.cpp  in order to accept lilypond dynamics

What are your opinions on that?

You can pass the goal as a string of course, as we discussed (at length) before.
You have to be exact about what strings you accept though because strings can contain anything (e.g., json wrapped base64 encoding of cat pictures...) and thus do not define an interface.
So if you do not want to support all of lilypond you should document examples you want to support above.
To illustrate various examples of features that I think are easy to support through lilypond or that you need to support to some level due to requirements, I wrote a (not-so-)small example using the first phrase of "all my ducklings".
A lot of the .ly file is for formatting and does not affect the sound, so the actual features are not that intimidating :rabbit2:
As I mentioned to @Flova and @scaredycode last week, I also wrote some code for myself to convert a lilypond string/file into a list of "audio-perception-style" NoteOnsets. That is basically what you would expect the music perception to output when you play the piece. It calls lilypond in the background and thus would support all of lilypond-syntax, but I don't know whether you want to use it at all or possibly restrict the syntax on purpose to make it easier to interpret.

I defined the action and a new message type.
Naming and definition strict to what you have written down there @scaredycode  https://github.com/UHHRobotics22-23/marimbabot/issues/111 .
The overall structure should be ready now just the TODOs, like creating a publisher and subscriber for information exchange between planning and action server, have to be implemented.


Forge Port

An official Forge port is unlikely to happen due to my time constraints, but many of my client side mods(including Explosive Enhancement) should work perfectly fine with Sinytra Connector.
https://modrinth.com/mod/connector
https://github.com/Sinytra/Connector


TypeError: argument 'mode': 'int' object cannot be converted to 'ImgColor'

I'm on windows, using python 3.12.4
i also tried using venv by running
py -m venv venv
.\venv\Scripts\Activate.ps1
pip install -r .\requirements.txt

Here is the full traceback
https://pastes.dev/ebezdKsTm3
Something that might be of interest, the version of the dependencies installed seems to be newer than the one on requirements.txt
Successfully installed chainner-ext-0.3.10 colorama-0.4.6 numpy-1.26.4 opencv-python-<IP_ADDRESS> pepeline-0.3.8 tqdm-4.66.4

I apologise for the oversight regarding this project. Lately, under the influence of a friend's persuasion, I have modified the reading API within the pepeline, which has resulted in minor issues across all my projects. You may either revert to using the 3.7 pepeline or allow me some time to rectify the situation.

fixed it


Preference styles are overriden by library layout resource

Hi Aidan,
I'm styling my preferences through styles.xml instead of custom layouts. The problem is since version 8.4.0 these styles are overriden by the library preference layout resource.
Is there any way I can avoid using the library layout without having to create a custom layout for my preferences? Or in other words, could it be possible to make the use of the library preference layout optional?

What version are you using? This was fixed in release <IP_ADDRESS>, the current is <IP_ADDRESS>.

To be more specific, the logic that prevents layouts from being overridden now is here:
https://github.com/afollestad/material-dialogs/blob/master/commons/src/main/java/com/afollestad/materialdialogs/prefs/PrefUtil.java#L22

No, my problem is different. I'm not using a custom layout. I'm using attributes in styles.xml for my preferences. The problem stays in <IP_ADDRESS>.
The problem is if I don't want to use the library layout, I'm forced to create a custom layout compulsorily. There's no other option. And setting a null custom layout doesn't help, because it overrides the styles too.
Maybe this could be bypassed with an attribute for the preference, indicating if we want to use the library layout or not.

+1

Added a app:useStockLayout="true" attribute which will force preferences to use the stock layout (not override the layout).

https://github.com/afollestad/material-dialogs/releases/tag/<IP_ADDRESS>

Thank you very much Aidan, I really appreciate it.


[feature request] Get last observation in trajectory with vecenv

At present, the stable-baselines vecenv implementations drop the last observation in each trajectory (i.e the one for which .step() returns done=True) & replaces it with an initial observation produced by .reset(). For instance, in DummyVecEnv, the end of a trajectory is handled as follows:
obs, self.buf_rews[env_idx], self.buf_dones[env_idx], self.buf_infos[env_idx] =\
    self.envs[env_idx].step(self.actions[env_idx])
if self.buf_dones[env_idx]:
    # drops observation for which `.step()` reported `done=True`
    obs = self.envs[env_idx].reset()
self._save_obs(env_idx, obs)

This seems like a reasonable design choice for many RL algorithms, but there are also combinations of environment & algorithm for which having all (s,a,s') transitions—including the last one—is useful. For example, GAIL-from-observations recovers an imitation policy by learning a reward function of the form r(s,s') on (state, next state) tuples that scores how "demonstrator-like" a trajectory is. If the last state in each trajectory is very informative (e.g perhaps the final screen in an arcade game says "you lost!" or "you won!"), then the imitation learning algorithm will find it much easier to learn an accurate reward function if it has access to the final (s,a,s') transition.
There are a few ways of giving users access to the last state in each rollout. One minimally-invasive way would be to add it to buf_infos under a descriptive name like "done_new_obs" whenever the episode terminates. Does this seem like a reasonable change?

:+1: on this issue -- it's a real problem when collecting data for imitation learning algorithms. In environments where the ground truth is a sparse reward, it makes it impossible for the model to accurately capture the reward.
I have a mild concern with modifying the infos dict, since consumers may not be expecting a new key. An alternative would be to add some property, obs_before_reset, and then samplers can if desired query this for environments with a done flag.

Modifying the infos dict is not usually a problem, as users should only look for certain keys, and adding more infos should not break things. As an example, some gym wrappers (like the Monitor wrapper) add some (undocumented) keys to the info dict.
However, for the name, I'm not sure what to choose:

done_new_obs is a bit confusing for me
obs_before_reset seems ok even though I'm not completely sure
last_observation would be also confusing
episode_end_observation ? seems also odd

Ah yes, I forgot that behavior of Monitor, probably OK for us to add keys then.
For the name, how about terminal_observation?

+1 for the name ;)


properly loading model

Patch Description
Fix model initialization for some of the OPT models.
Issue:
Models up to 13B get loaded but run into an error about a half/float mismatch, while 30B model runs fine (see discussion: https://fb.workplace.com/groups/gogogptzusers/permalink/762243555001669/).
Debugging:
After loading model from /data/gpt-z/models/gptz/1.3B/consolidated_mp_2/consolidated.pt (azure), and print out [p.dtype for p in model.parameters()], it shows a mix of  torch.float16 and torch.float32.
Found that cfg.model.tensor_parallel_init_model_on_gpu = False in 1.3B model (True in 30B model), so  the model is not properly initialized and it fails silently due to dtype auto cast in model.load_state_dict.
This patch fix it.
Testing steps
Able to launch internactive_cli.py and interactive_hosted.py with the following model path:
f"--path /data/gpt-z/models/gptz/2.7B/consolidated_mp_1/consolidated.pt",
f"--path /data/gpt-z/models/gptz/30B/consolidated_mp_4/reshard.pt",
f"--path /data/gpt-z/models/gptz/30B/consolidated_mp_2/consolidated.pt",
f"--path /data/gpt-z/models/gptz/1.3B/consolidated_mp_2/consolidated.pt",

I would also want to add "--fp16", "--memory_efficient_fp16" (and later on '"--bf16") in constant.py to keep model intialization the same as training. But it is not needed for current fix.


Cannot import API

Cannot import the stub APi at /examples/apis/posts.json. It fails with this response:
HTTP/1.1 400 Bad Request
Content-Length: 27
Content-Type: application/json
Date: Wed, 05 Apr 2017 22:43:03 GMT
Vary: Accept-Encoding
"api definition not found"
Reproduction Steps:

Start all docker containers with: docker-compose up -d
Create valid JWT token
Submit this request from within the janus-master folder via CLI:

http POST <IP_ADDRESS>:8081/apis "Authorization:Bearer eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJleHAiOjE0OTE0MzM3MTIsImlhdCI6MTQ5MTQzMDExMiwiaWQiOiJhZG1pbiJ9.CfNK3mxh3yArgrDXBLpYArLqPU6PywIDCxBdNEeSQUA" "Content-Type: application/json" < examples/apis/posts.json
Expected behavior:
Successful addition of jsonplaceholder API
Observed behavior:
API definition not found error.
**Janus version: latest version
**OS and version: Mac OSX 10.11.6

Hi @cosjef,
I couldn't reproduce the behavior that you mentioned. What kind of storage for your configs are you using? File System or MongoDB?
If it is file based you won't be able to use the write operations of the REST API, which means, the API will be in Read Only mode. So all changes that you do will be on a file.
If you are using MongoDB as your database, then after adding the configuration you MUST restart the gateway so the changes are applied. There is a PR opened #120 which will implement the hot reload feature and you won't have to reload it anymore.
I hope this help you.

I am using MongoDB as the storage system. It looks like the gateway does not accept the api definition at all. Are you saying that even in the presence of the error, I should restart the gateway? Do you know what validation in the code is causing this error?

Sorry I didn't mean that.
I tried to execute here the same request that you've mentioned and it worked....
I've tried on master, maybe there is something wrong with the docker image, I'll try to do the same thing there.
Just to help me a bit, what version are you using? 2.2.0-rc2?

I was running 2.2.0-rc.1 via direct download from Github master branch. I just downloaded janus-2.2.0-rc.2, unzipped the file and navigated into the directory. Then started the containers with "docker-compose up -d" and all started. Then got a JWT token using this command: http -v --json POST <IP_ADDRESS>:8081/login username=admin password=admin
After obtaining a JWT, I used this command to try adding the API: http -v POST <IP_ADDRESS>:8081/apis "Authorization:Bearer eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJleHAiOjE0OTE0ODY1NzcsImlhdCI6MTQ5MTQ4Mjk3NywiaWQiOiJhZG1pbiJ9.4h4R6zy_nOemLiRCPDniLxCVxPWzhEwoch5eATqp9V0" "Content-Type: application/json" < examples/apis/posts.json
Same result:
POST /apis HTTP/1.1
Accept: application/json, /
Accept-Encoding: gzip, deflate
Authorization: Bearer eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJleHAiOjE0OTE0ODY1NzcsImlhdCI6MTQ5MTQ4Mjk3NywiaWQiOiJhZG1pbiJ9.4h4R6zy_nOemLiRCPDniLxCVxPWzhEwoch5eATqp9V0
Connection: keep-alive
Content-Length: 1076
Content-Type: application/json
Host: <IP_ADDRESS>:8081
User-Agent: HTTPie/0.9.8
{
"active": true,
"allowed_ips": [],
"cors_meta": {
"domains": [
""
],
"enabled": true,
"exposed_headers": [
"X-Debug-Token",
"X-Debug-Token-Link"
],
"methods": [
"GET",
"POST",
"PUT",
"PATCH",
"DELETE"
],
"request_headers": [
"Origin",
"Authorization",
"Content-Type"
]
},
"name": "Posts",
"oauth_server_name": "local",
"proxy": {
"append_path": false,
"enable_load_balancing": false,
"hosts": [
"hellofresh."
],
"listen_path": "/posts/*",
"methods": [
"GET"
],
"preserve_host": false,
"strip_path": true,
"upstream_url": "https://jsonplaceholder.typicode.com/posts"
},
"rate_limit": {
"enabled": true,
"limit": "10-S"
},
"use_basic_auth": false,
"use_compression": false,
"use_oauth2": false
}
HTTP/1.1 400 Bad Request
Content-Length: 27
Content-Type: application/json
Date: Thu, 06 Apr 2017 12:49:57 GMT
Vary: Accept-Encoding
"api definition not found"


Publish this as a nuget package on Episerver nuget feed

It would be so much easier if this was a nuget package on episervers feed, it could be marked as beta or something to show that is is not a official package

Sure, maybe a better title and description on the NuGet package is enough


Issue #1232 Resolved

Added documentation and folder for Open-CV projects Gait Recognition and Virtual Painter Projects

@sanjay-kv Please add labels to the PR


Correct query loss when using parse_qsl to dict

NOTE: I noticed this when doing a fairly comprehensive source check to find the httplib2 footprint (for #128).
The issue is that repeated params could just be dropped:
>>> from six.moves import urllib
>>> params_list = urllib.parse.parse_qsl('a=1&a=2')
>>> params_list
[('a', '1'), ('a', '2')]
>>> dict(params_list)
{'a': '2'}

This LGTM, but I wonder if there's not already something that handles this well?

I just did git grep parse_q. It's crazy it was used incorrectly in 4 different places.

@jonparrott I want to hold off on the merge until @nathanielmanistaatgoogle weighs in

@dhermes good plan. This seems like hackery I'm not sure I like.

This seems like hackery I'm not sure I like.

What is "this"?

Doing urlparsing and such.

Yeah I dunno. You can file an issue to follow-up?

Just more to think about before we close #128 I think.

Currently typing one-handed w/baby in my arms, can you add a checkbox about it in the long desc. for #128 for me?

I read that as "typing with baby arms". :D
Will do.

@nathanielmanistaatgoogle PTAL.
In particular, we should resolve what we think the actual goal of all those dict(parse_qsl(...)) calls were.

Agreed that this needs a deeper look; probably site-by-site (luckily there are few). Replace-except-if-the-value-is-a-list-of-two-or-more-then-add is just weird semantics.
If it's the case that all sites in the library are correct with replace semantics, then our library-private _helpers module should make available a helper function that implements replace semantics.
If it's the case that all sites in the library are correct with add-to semantics, then our library-private _helpers module should make available a helper function that implements add-to semantics.
If it's the case that some sites in the library are correct with replace semantics and some are correct with add-to semantics, then our library-private _helpers module should make available a helper function that implements replace semantics and a helper function that implements add-to semantics.
I seriously doubt that it's the case that there's a site in the library where the correct semantics are replace-except-if-the-value-is-a-list-of-two-or-more-then-add.
Am I making sense?

Yes. But the issue with replace is if the original had multiple values, then what does replace mean? Which is why I suggested we first verify that each parameter shows up exactly once, then replace.
i.e. a=1&b=2 replace b=3 (or c=3) is easy but what if the original was b=1&b=2

For private helper methods, replace means what we implement it to mean. I could see it going one of two ways: either "replace" semantics mean "associate this key with this value, no matter whether zero, one, two, or more values were associated with the key previously" or "replace" semantics mean "associate this key with this value, and if there were zero values with the key previously, that's fine, and if there was one value with the key previously, that's also fine, but if there were two or more values associated with the key previously, that's a problem so raise an exception". While either of these choices would be acceptable to be blessed as correct, I suspect it's the first that is going to be more useful and applicable in our codebase. Is my suspicion correct?

@nathanielmanistaatgoogle Hopefully this works. Don't forget to remind me to squash on LGTM

Looks good (after out-of-band discussion); squash and merge.


Communication resource preparer for Communication CLI extension tests

Description

PR is for adding communication resource preparer which will be used in AZ communication CLI tests.
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Communication

@ravithanneeru Please fix CLI Style Check issue. Thanks

@ravithanneeru Please fix CLI Style Check issue. Thanks

Done. Seems all jobs are successful now.


Doc: fix typo in example README.md files

There is a small typo made in the examples for running the cargo test suite. --nocaputre is not a valid option, here's a simple fix :rocket:
Checklist

[ ] Updated guide with pertinent info (may not always apply).
[x] Squash down commits to one or two logical commits which clearly describe the work you've done.
[ ] Unittest is a friend:)

This change is

Thank you!


java.lang.Exception: Could not get session-init packet, maybe the MUC has locked

INFO: waitForInitPacket
java.lang.Exception: Could not get session-init packet, maybe the MUC has locked
This is the exception i'm getting when i'm executing the jar with all the properties set. It joins the MUC perfectly but since waitForInitPacket is synchronized, it halts at this stage with the specified exception.
Jicofo throws exception as org.jitsi.jicofo.UnsupportedFeatureConfigurationException: Participant doesn't support bundle, which is required
Could you please help on this??

We need to add two features for any client as per Videobridge2...  jicofo looks up for the features of the client/participant supports. It should support RTCP_MUX and RTP_BUNDLE i.e, urn:ietf:rfc:5761 and urn:ietf:rfc:5888. Then only jicofo allows the client/participant to join the room


The controller should fail if it's RBAC is misconfigured

The controller tries to access blueprints and actionsets using the watcher callbacks. If there is an access issue, these watcher goroutines will go into an infinite fail loop.
For example:
kubectl logs myrelease-kanister-operator-77dcff8694-5sz5d
time="2018-03-27T18:35:12Z" level=info msg="Getting kubernetes context"
E0327 18:35:13.178643       1 reflector.go:205] github.com/kanisterio/kanister/vendor/github.com/rook/operator-kit/watcher.go:76: Failed to list *v1alpha1.Blueprint: the server could not find the requested resource (get blueprints.cr.kanister.io)
E0327 18:35:13.178720       1 reflector.go:205] github.com/kanisterio/kanister/vendor/github.com/rook/operator-kit/watcher.go:76: Failed to list *v1alpha1.ActionSet: the server could not find the requested resource (get actionsets.cr.kanister.io)

We should check to make sure we have access to these synchronously and shutdown the controller if we cannot get actionsets.

This specific RBAC issue has been fixed.


Support nativeWidth and nativeHeight image properties for lazy loading

Hello deno-dom Team,
it would be great if you could provide nativeWidth and nativeHeight properties on images.
This would allow to implement width and height on lazy loaded images in SSR/SSG.
e.g. in Lume SSG I would like to use following script to add lazy loading with intrinsic width and height of images during server side processing:
    const images = page.document.getElementsByTagName('img') // page.document uses deno-dom
    Array.from(images).forEach((image) => {
        image.setAttribute('loading', 'lazy')
        image.setAttribute('width', image.naturalWidth) // undefined
        image.setAttribute('height', image.naturalHeight) // undefined
    })

In case I run it in the browser like this it works and finds the sizes correctly. I guess deno-dom does not yet support it:
    const images = document.getElementsByTagName('img')
    Array.from(images).forEach((image) => {
        image.setAttribute('loading', 'lazy')
        image.setAttribute('width', image.naturalWidth) // defined
        image.setAttribute('height', image.naturalHeight) // defined
    })

King regards
Tobias

I can implement it, but it will only return 0. Deno DOM won't attempt to determine the size of any images.

Actually, I can't implement it yet because Deno DOM doesn't have HTML* constructors implemented for all the specific types... i.e. in this instance we'd need HTMLImageElement, but until that's implemented I can't add the naturalWidth/naturalHeight properties...

Hi @tobiasschmidt89 ,
Did you find a workaround? Running into the same issue. :-)


[TASK] Update document to reflect on Volume Attachment Recovery Policy feature deprecation

What's the task? Please describe.
Update document to reflect on Volume Attachment Recovery Policy feature deprecation.
Describe the items of the task (DoD, definition of done) you'd like.

[x] documentation updated

Additional context
https://github.com/longhorn/longhorn/issues/2060
https://longhorn.io/docs/1.1.2/high-availability/node-failure/#volume-attachment-recovery-policy

Close it since it's a doc change only.


Menu example

I'd like to construct a menu for my app. I don't see any examples or docs for this, so I guess I'll have to figure it out from the code.
So my questions are:

any pointers to help me (even on fltk with C/C++)?

would you like a Menu example (in a PR) if I can figure it out?

Hello,
There's an example of a menu bar in the editor example here:
https://github.com/MoAlyousef/fltk-rs/blob/master/fltk/examples/editor.rs
It uses a SysMenuBar (to get MacOS menu bars), but you can use a MenuBar directly. The api is similiar for both. To add items you can use MenuBar::add() or MenuBar::add_emit(), the latter takes a sender and a message, while the former takes a boxed closure in place of both, both methods are part of the MenuExt trait:
https://docs.rs/fltk/0.8.5/fltk/prelude/trait.MenuExt.html
And yes I would love to have more examples in the repo :)


Access control gate API calls

This builds on the playback endpoint to add in a middleware which makes the Gate API calls to confirm whether playback is allowed or not.
The other main change is serving both manifests and media from a single endpoint, this was done to be able to serve static mp4s, it would've been too messy to try to rewrite these urls with a different path before hitting the playback endpoint here. So now I check the content-type and file extension to determine whether the file is a manifest.
Improvements to come

range requests

Marked as draft until https://github.com/livepeer/catalyst/pull/437 is merged and the dependency is updated here. Still ready for review though, no other changes I'm aware of.


[resolver] Fix issue with wheel when using --index-url option

Fixes #864

@Djailla can you merge latest master and push to this branch? That should get the tests here passing. Thanks!


opengl support?

hello do you plan to add opengl support to vulkan?
it would greatly help with my DXVK opengl implementation
and save me A LOT of work
many thanks

marcel

thank you for making such a good graphics api

i need to support older hardware

Can you elaborate? I'm not sure what you're actually asking for.
These are just examples for the Vulkan API, and not the Vulkan API itself.

hello there
ah I see this is just example for vulkan
you do not make vulkan api yourself I infer?
if so whom should I make my request of????

Nope, I'm just a member of the Vulkan Advisory Panel. The API itself is maintained by Khronos, the official repository can be found at https://github.com/KhronosGroup/Vulkan-Docs/


Feature request: Responsive design for repositories tab

At present it is a fixed width, but if the repo name and path on file system are too long, the text gets cut off.

Done. You can download the latest CI build from Github Action.