Email Tasks for Success and Failures

When a DAG completes or on errors, email the cataloger that the Sinopia resource was successfully imported into the ILS with either the CATKEY (Symphony) or the HRID (FOLIO). We should also email the cataloger if the DAG fails along with any context about the failed task(s).
We should use AWS SES as we are already doing for Sinopia email.

success case addressed by #64, filed #65 for dealing with task failure


chore: add .gitignore

Signed-off-by: hsgwa<EMAIL_ADDRESS>This is copy of https://github.com/tier4/CARET_analyze/blob/main/.gitignore

@hsgwa
Is it reasonable if .gitignore file is synchronized among repositories around CARET?

@takam5f2
That's make sense.
Though modifying .gitignore is not so frequently, adding sync file is better.

Is it reasonable if .gitignore file is synchronized among repositories around CARET?

Since CARET has cpp and python packages, caret_common/.gitignore should have cpp and python supported .gitignore.
I'll send PR to caret_common to add items in .gitignore first.
https://github.com/tier4/caret_common/blob/main/.gitignore
and then, I open PR to sync .gitignore from caret_common to each packages.


InvalidOperationException in AuthenticationMiddleware while AuthenticationSchemeProvider.RemoveScheme

In case of simultaneous execution AuthenticationSchemeProvider.RemoveScheme from background Task and any request, which uses AuthenticationMiddleware may get
System.InvalidOperationException: Collection was modified; enumeration operation may not execute.
   at System.ThrowHelper.ThrowInvalidOperationException(ExceptionResource resource)
   at System.Collections.Generic.List`1.Enumerator.MoveNextRare()
   at System.Collections.Generic.List`1.Enumerator.MoveNext()
   at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware.<Invoke>d__6.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Cors.Infrastructure.CorsMiddleware.<Invoke>d__8.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.GetResult()
   at IdentityServer4.Hosting.BaseUrlMiddleware.<Invoke>d__3.MoveNext()

By default AuthenticationMiddleware using AuthenticationSchemeProvider.GetRequestHandlerSchemesAsync() for retrieving all schemes and enumerate results via foreach.
Problem occurs when removing scheme from _requestHandlers at the moment when enumeration in process from foreach (without any locks)
Microsoft.AspNetCore.Authentication.dll version 2.1.1
Microsoft.AspNetCore.Html.Abstractions.dll 2.1.1
dotnet info:
.NET Core SDK (reflecting any global.json):
 Version:   2.1.403
 Commit:    04e15494b6

Runtime Environment:
 OS Name:     Windows
 OS Version:  10.0.17134
 OS Platform: Windows
 RID:         win10-x64
 Base Path:   C:\Program Files\dotnet\sdk\2.1.403\

Host (useful for support):
  Version: 2.1.5
  Commit:  290303f510

.NET Core SDKs installed:
  2.1.401 [C:\Program Files\dotnet\sdk]
  2.1.402 [C:\Program Files\dotnet\sdk]
  2.1.403 [C:\Program Files\dotnet\sdk]

.NET Core runtimes installed:
  Microsoft.AspNetCore.All 2.1.3 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.All 2.1.4 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.All 2.1.5 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.App 2.1.3 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 2.1.4 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 2.1.5 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.NETCore.App 2.1.3 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.1.4 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.1.5 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]

Hmm, we lock on add and remove, but not on enumeration. This List needs to be a ConcurrentQueue.

Yeah, should we try to fix this in 2.2 rtm?

Yeah, it looks like a simple drop in replacement.

Maybe just add the lock.

The enumeration takes place in a different class (the auth middleware) that doesn't know about the lock.

Queue isn't a straight replacement since we support Remove, while queue only supports removing at ends :/  We could use ConcurrentDictionary but then we'd have a slight change in behavior that GetAllSchemes no longer would return them in the order they were registered...
If we do this in 3.0 we could just accept that slight behavior change maybe?  Main diff would be the order of the request processing schemes would potentially change (unless we added some complexity to track the scheme order)

We should prob discuss this at next security triage, tagging @Eilon @blowdart since this is issue isn't in security repo

Maybe the short term fix for 2.2, is we just lock and copy the _schemes to another enumerable we return instead of directly returning the values?

Something like this maybe?
        public virtual Task<IEnumerable<AuthenticationScheme>> GetAllSchemesAsync()
        {
            IEnumerable<AuthenticationScheme> copy = null;
            lock (_lock)
            {
                copy = _schemes.Values.ToList();
            }
            return Task.FromResult(copy);
        }

That's quite expensive to do on every request.

We can cache the copy maybe, and only copy when it changes which shouldn't be often

The copying would only happen when adding/remove schemes which is usually done only during startup, so it wouldn't be noticeable really.  Copying a small list inside the lock shouldn't be a big deal since its already inside of a super slow lock right?

Ah OK

I'll send a PR so its more clear what the fix would be

@Eilon do we want this for 2.2?

Sounds fine to me. The PR is signed off so that looks alright.

Fixed via https://github.com/aspnet/HttpAbstractions/pull/1052


Is there a plan to add vim-lsp as a source?

I guess vim-lsp is not a priority for you to add as a source, still I would like to know if you plan to add it anytime soon.
I use ccls with vim-lsp. I tried switching to nvim-lspconfig, but for some reason my cpu usage shoots through the roof when I use the built-in lsp feature. Was forced to switch back to vim-lsp.
I would love to see nvim-cmp support vim-lsp.
Thank you.

I use ccls with vim-lsp. I tried switching to nvim-lspconfig, but for some reason my cpu usage shoots through the roof when I use the built-in lsp feature. Was forced to switch back to vim-lsp.

Which version of neovim are you on? This should not be the case (sorry to hijack the issue).

Ah, you're free to override the root detection ordering in your setup call to whatever logic would detect the appropriate root for your codebase.

No, there's always a minimal amount of logs by default (start stop client and errors)

@dmitmel is implemented this. Thank you!


Add files via upload

@ThanhCongTruong file/lib/domains/vn/edu/ufm.txt: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.

@ThanhCongTruong Thank you for your patience while waiting for a review and processing your request. Pull request merged.


Override broken old versions of azure-ai-ml when running storage regression tests

@jalauzon-msft

Thanks for handling this!

Sorry for the delay 👎

/check-enforcer override


Filter with multiple languages

First, the project is very useful! Thanks for creating it! However, it's kinda embarrassing that we can only search on one language once a time.
Can we add multiple language support so that we can search based on various combination of languages? Like python and c++

Thanks for bringing this up, it would be a nice feature! It can be achieved by specifying multiple languages with +language:a+language:b when querying GitHub. We can possibly allow --lang to be specified multiple times or with commas: --lang=python --lang=cpp or --lang=python,cpp.

Hi! My friend and I think this is a very good start for our contribution to GitHub. Could we work on this? :)

I understand that I have asked to solve two issues, but we also have two people here. So it should be not that hard. Hope you can assign us the issues, thx!

sure

Hello @hedyhli ! Could I work in this issue?

@hedyhli Hello, I've submitted a pull request to enable filtering with the combination of languages and spoken languages.
Currently, we are able to use -l Python+JavaScript and -S en+zh.

Hello @hedyhli ! Could I work in this issue?

Hey @larissalages thanks for you interest but I'll let @fgsepter have a go at this first since they asked first 😄

This work still open? Can I work on this?

Do you still need help? I can pick it up

Closing as implemented (#116).


Everything appears very large (Linux/X11)

Hi,
When I start any DlangUI project, all the controls are particularly large on my system. Other GUI programs work fine.

I'm assuming this isn't supposed to happen. Does anyone know what might cause it?

High DPI screen is detected.
Not sure what to do with it.

The screen is 140dpi.
How does DlangUI detect this? Maybe I can change that

Dlangui reads screen DPI using SDL2 calls.
Default DPI is 96. So, for 140 everything will be scaled.

possible workaround - allow application to override system DPI.
E.g. calling overrideSystemDPI(96) would disable scaling.
overrideSystemDPI(0) would reenable usage of detected DPI.
As well it will be possible to choose custom UI scaling by application - e.g. somewhere in settings.

If you really want to stick to X11 dlangui version (which I would not recommend, since this platform implementation is currently incomplete in dlangui, e.g. getting clipboard contents does not work) you may try to use XRandr calls. See https://lists.x.org/archives/xorg-devel/2017-April/053430.html

What alternative do I have if I want to use Dlangui on Linux?

SDL2. This is default configuration.

My apologies. I meant X11 as in the underlying gfx server, as opposed to Wayland. I'm using the SDL back end for Dlangui. I wasnt aware there was an X11 one.

Check if SDL2 version is at least 2.0.4. That's when SDL_GetDisplayDPI was added. You'll get message "SDL_GetDisplayDPI is not found: cannot detect screen DPI" if this function is missed.

Implemented overriding of detected screen DPI.
To disable scaling, add to your app UIAppMain()
 overrideScreenDPI(96);

Perfect, thank you! I'll test this later today.


Add a new loglevel between "info" and "debug" and change ConfigShell.run_cmdline() to log "Running command line" message to it

There's a HUGE difference between the info and the debug loglevels. The latter is virtually unusable -- it's waaay over the top.
One thing I like about it though is the ability to echo back commands as they are run. To make this more feasible, I'm going to add a loglevel between info and debug and modify ConfigShell.run_cmdline() to use that loglevel.

Merged in https://github.com/open-iscsi/configshell-fb/pull/38


Removes the bug of adding categories

What was happening:
We are using the same DOM element .delete-popover to represent the popover that appears on right-clicking a note. So, whlie creating a new category, if we type in the input field specified and press <Enter>, the category is registered and the name of the added category remains as text in the input field (See the image posted by @fonorobert on #175).
So, on right-clicking any note, since we are using the same pop-over, instead of the placeholder text add category, the name of the added category appears.
File Changes:
Line no. 33 defines a function to clear the input field and line no. 46 calls the function on pressing <Enter>.
Line no. 23:
If we type the name of a note without pressing <Enter> and switch to a different note, still the name typed should not persist. So on focusout event, we need to call the cleanField(e) function.
This solves #175.

Can close #175 now :)


Fixtures

Hi, I had some time on an plane and worked on some fixtures. For details, please see commit messages. Feedback welcome

Thanks for the review. I've updated the PR and pushed just as a safety measure before the flight. I'll finish the work one home :)

So, this should be everything now. Feel free to review and point out things to be fixed. If all looks good, please merge :)

Many thanks! :+1:


added python script to download and convert AWS TS file to MP3

@pastorep what should happen with this PR now?


Add backend to full page view of Upcoming Grants table

@anigonzales this issue seems like it should be closed? Is it done?

@anigonzales this issue seems like it should be closed? Is it done?

yes, i'll close this


AMBARI-23293. Fix the following : 1. Host Component deletes are broken 2. Use resourceType for getting resources from Mpack Modules.

What changes were proposed in this pull request?
AMBARI-23293. Fix the following :

Host Component deletes are broken
Use resourceType for getting resources from Mpack Modules.

1. Host Component deletes after fix:
DELETE http://:8080/api/v1/clusters/c1/hosts//host_components/13}
{
  "deleteResult" : [
    {
      "deleted" : {
        "key" : "component_id: 13"
      }
    }
  ]
}

2. Use resource Type in for getting resources from Mpack Modules.
Reason: When we are creating a Service, Host Component and Service Component when name provided is not same as the type, we need to use the ResourceType to refer the mpack to get the resource component from stack.
Eg:
 {
      "ServiceInfo":{
         "service_group_name":"dev",
         "service_type":"ZOOKEEPER",
         "service_name":"ZOOKEEPER_dev"
      }
   }

How was this patch tested?

Deployed cluster with changes. Worked fine.
Create Multi SG, Service, Service and Host Components. Worked fine. Used Postman Scripts from : LINK

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/1244/
Test FAILed.
Test FAILured.


[wrt] Fix duplicated test purpose issue

Fix duplicated test purpose issue for internetstdmanu, rt
, rtcoremanu and sharedmodemanu

LGTM


dat sync crashes with ENOSPC

trying to sync a folder with  38726 folders and 42024 files on linux fails:
$ dat sync
Sharing dat: 566 files (442 MB)

0 connections | Download 0 B/s Upload 0 B/s

Metadata created for 566 of 6566 files (2.9 MB/s)
(Calculating file count...)
ADD: pdfs/example.pdf (1.2 MB)

Ctrl+C to Exit
fs.js:1445
    throw error;
    ^

Error: watch /data/pdfs/123/456 ENOSPC
    at exports._errnoException (util.js:1020:11)
    at FSWatcher.start (fs.js:1443:19)
    at Object.fs.watch (fs.js:1470:11)
    at /usr/local/lib/node_modules/dat/node_modules/recursive-watch/index.js:106:18
    at FSReqWrap.oncomplete (fs.js:123:15)

import continues to work if watch is disabled:  dat sync --watch=false
Versions:
$ dat -v
13.9.0

$ node -v
v6.11.4

You've hit an is limit. The error means no space on the device, but I
suspect it is the number of files that dat is watching. See solutions in
this thread https://github.com/gruntjs/grunt-contrib-watch/issues/236
On Sun, Oct 15, 2017, 9:21 AM bit<EMAIL_ADDRESS>wrote:

trying to sync a folder with 38726 folders and 42024 files on linux fails:
$ dat sync
Sharing dat: 566 files (442 MB)
0 connections | Download 0 B/s Upload 0 B/s
Metadata created for 566 of 6566 files (2.9 MB/s)
(Calculating file count...)
ADD: pdfs/example.pdf (1.2 MB)
Ctrl+C to Exit
fs.js:1445
throw error;
^
Error: watch /data/pdfs/123/456 ENOSPC
at exports._errnoException (util.js:1020:11)
at FSWatcher.start (fs.js:1443:19)
at Object.fs.watch (fs.js:1470:11)
at /usr/local/lib/node_modules/dat/node_modules/recursive-watch/index.js:106:18
at FSReqWrap.oncomplete (fs.js:123:15)
import continues to work if watch is disabled: dat sync --watch=false
Versions:
$ dat -v
3.9.0
$ node -v
v6.11.4
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/datproject/dat/issues/880, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAwkuo0SdY95l0pT6YcKZD0eaIOBvC25ks5ssiMQgaJpZM4P5yCi
.

looks like it was indeed hitting that limit
cat /proc/sys/fs/inotify/max_user_watches
8192

testing with higher limit now:
echo 524288 > /proc/sys/fs/inotify/max_user_watches

Each used inotify watch takes up 540 bytes (32-bit system), or 1 kB (double - on 64-bit)
would be nice to figure out a way to not use that much memory for larger repos.

I'm not sure there is another way we can do watching without all these file descriptors, at least with the default node implementation - see the node caveats. If someone knows of recursive watching in linux without that problem, that'd be great.
Otherwise the solution is to not live-watch files. You can do one time imports but not the live imports when files change.

You might catch that error and fall back to a poll mode

On Oct 15, 2017, at 11:46 AM, Joe Hand<EMAIL_ADDRESS>wrote:
I'm not sure there is another way we can do watching without all these file descriptors, at least with the default node implementation - see the node caveats. If someone knows of recursive watching in linux without that problem, that'd be great.
Otherwise the solution is to not live-watch files. You can do one time imports but not the live imports when files change.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Introducing module declaration file

Introducing index.d.ts file in order to declare the type definitions for SteamSignIn module

:+1: Thanks


New connection pool & connection

@henryanand @shrutip @enisoc

LGTM

LGTM but is the unit test failure a true failure?

The unit test is supposed to be disabled. It's manual only right now. I'll disable it.


Get Current TraceId in ApolloServerPlugin - GraphQLInstrumentation

Hello!
I have configured GraphQLInstrumentation as per below, and it's working fine, I can see my traces in Jaeger UI for example:
import {
    context,
    Span,
    trace
} from "@opentelemetry/api";

const provider = new NodeTracerProvider({
    resource: Resource.default().merge(new Resource({
        "service.name": "my-service"
    })),
});

trace.setGlobalTracerProvider(provider);

provider.addSpanProcessor(new BatchSpanProcessor(new JaegerExporter()));

// Register the provider to begin tracing
provider.register();

const graphInstrumentation = new GraphQLInstrumentation({
    responseHook: (span: Span, data: ExecutionResult) => {
        // get current traceId
        const activeContext = context.active();
        const spanContext = trace.getSpanContext(activeContext);
        console.log("TraceId", spanContext?.traceId); // THIS WORKS FINE, I GET  THE TRACE ID
    },
});

graphInstrumentation.setTracerProvider(trace.getTracerProvider());

registerInstrumentations({
    instrumentations: [
        graphInstrumentation,
        new HttpInstrumentation({
            requireParentforIncomingSpans: true,
        }),
    ],
});

In my response hook the spanContext contains valid traceId and spanId. However, if I try to do the same in an ApolloServerPlugin, the TraceId is 0000000...
import {
    context,
    Span,
    trace
} from "@opentelemetry/api";

const exceptionPlugin = {
    requestDidStart(requestContext: RequestContext): Promise < GraphQLRequestListener > {
        return Promise.resolve({
            didEncounterErrors(
                requestContext: GraphQLRequestContextDidEncounterErrors < BaseContext >
            ) {
                const activeContext = context.active();
                const spanContext = trace.getSpanContext(activeContext);
                console.log("TraceId", spanContext?.traceId); // HERE IT PRINTS 00000000...

                return Promise.resolve();
            }

        });
    }
}

new ApolloServer(
    // several config omitted for simplicity
    plugins: [exceptionPlugin]
);

What would be the correct way to get the current trace here? I tried several apollo hooks, such as requestDidStart, willSendResponse, executionDidEnd, but I'm not able to get the current traceId in any of them.
My apollo plugin works as a global exception handler in which I send error messages, and I wanted to include the traceId for the current operation.
Thanks for the great libraries!!!

pls share part of package.json with all otel dependencies

hmm I'm a bit surprised all works anyway as those are mixed version and compatibility pls check this
https://github.com/open-telemetry/opentelemetry-js#compatibility-matrix
can you please update to latest

@obecny I upgraded it so it looks like this:
"@opentelemetry/api": "1.0.2",
"@opentelemetry/core": "^1.0.1",
"@opentelemetry/exporter-jaeger": "^1.0.0",
"@opentelemetry/instrumentation-graphql": "^0.26.0",
"@opentelemetry/instrumentation-http": "0.26.0",
"@opentelemetry/node": "^0.24.0",

So now I even downgraded @opentelemetry/api so it's 1.0.2 the exact same version used internally by @opentelemetry/instrumentation-graphql, but still not able to get the traceId.

This is most likely a problem with the graphql instrumentation not the API. Context is probably not being set when the requestDidStart callback is called. I'm going to transfer this plugin to the contrib repo where that instrumentation resides.


UNHANDLED EXCEPTION!!! TRegExpr(comp): Urecognized Modifier (pos 10)

^(?!([0-9])\1{9})[0-9]{10}$
1111111111 - not ok
1111111112 - ok

It does not support look-ahead
https://regex.sorokin.engineer/en/latest/faq.html#regular-expressions-with-do-not-work

Duplicates
https://github.com/andgineer/TRegExpr/issues/1


FancyZones Editor crashing when RTSS running with OSD enabled

Environment
Windows build number: Version 10.0.18362.356
PowerToys version: 0.11.0
PowerToy module for which you are reporting the bug (if applicable): FancyZones

Steps to reproduce

Have RivaTuner Statistics Server installed and running with On-Screen Display turned on (any
"application detection level" > none)

Expected behavior
the FancyZones layout selection window should open along with the layout preview
Actual behavior
FancyZones does not open, the process crashes and produces a Windows event log similar to this:
Faulting application name: FancyZonesEditor.exe, version: <IP_ADDRESS>, time stamp: xxxx
Faulting module name: d3d9.dll, version: 10.0.18362.329, time stamp: xxxx
Exception code: 0xc0000005
Fault offset: 0x00000000000ac416
Faulting process id: xxxx
Faulting application start time: xxxx
Faulting application path: C:\Program Files\PowerToys\modules\FancyZonesEditor.exe
Faulting module path: C:\WINDOWS\SYSTEM32\d3d9.dll
Report Id: xxxx

Occasionally the white overlay will flash very quickly but no windows will open
Workaround
In  RTSS, add a custom rule to disable the OSD specifically for FancyZones.exe
or
disable the OSD altogether
Note to PowerToys maintainers:
I created this issue with the intent to document the interaction between the 2 applications for people who are experiencing FancyZones crashes. If this is deemed out of scope and closed I take no issue with that.

Hi @cakekindel
thank you for reporting the crash.

Sorry, I cannot reproduce. Tried both master and the released PowerToys with RTSS 7.3.2. The editor works fine while the OSD is visible in games.

Closing because we cannot reproduce the issue. If it still happens, feel free to reopen the issue.


read osm.pbf

I tried this
import geopandas as gpd
url = "https://download.geofabrik.de/europe/liechtenstein-latest.osm.pbf"
gdf = gpd.read_file(url, engine="pyogrio")

returned this error:
/opt/conda/lib/python3.11/site-packages/pyogrio/raw.py:137: UserWarning: Layer 'b'points'' does not have any features to read
  result = ogr_read(

---------------------------------------------------------------------------
FeatureError                              Traceback (most recent call last)
Cell In [3], line 1
----> 1 gdf = gpd.read_file(url, engine="pyogrio")

File /opt/conda/lib/python3.11/site-packages/geopandas/io/file.py:271, in _read_file(filename, bbox, mask, rows, engine, **kwargs)
    268             from_bytes = True
    270 if engine == "pyogrio":
--> 271     return _read_file_pyogrio(filename, bbox=bbox, mask=mask, rows=rows, **kwargs)
    273 elif engine == "fiona":
    274     if pd.api.types.is_file_like(filename):

File /opt/conda/lib/python3.11/site-packages/geopandas/io/file.py:427, in _read_file_pyogrio(path_or_bytes, bbox, mask, rows, **kwargs)
    424     kwargs["read_geometry"] = False
    426 # TODO: if bbox is not None, check its CRS vs the CRS of the file
--> 427 return pyogrio.read_dataframe(path_or_bytes, bbox=bbox, **kwargs)

File /opt/conda/lib/python3.11/site-packages/pyogrio/geopandas.py:149, in read_dataframe(path_or_buffer, layer, encoding, columns, read_geometry, force_2d, skip_features, max_features, where, bbox, fids, sql, sql_dialect, fid_as_index, use_arrow, **kwargs)
    146 path_or_buffer = _stringify_path(path_or_buffer)
    148 read_func = read_arrow if use_arrow else read
--> 149 result = read_func(
    150     path_or_buffer,
    151     layer=layer,
    152     encoding=encoding,
    153     columns=columns,
    154     read_geometry=read_geometry,
    155     force_2d=force_2d,
    156     skip_features=skip_features,
    157     max_features=max_features,
    158     where=where,
    159     bbox=bbox,
    160     fids=fids,
    161     sql=sql,
    162     sql_dialect=sql_dialect,
    163     return_fids=fid_as_index,
    164     **kwargs,
    165 )
    167 if use_arrow:
    168     meta, table = result

File /opt/conda/lib/python3.11/site-packages/pyogrio/raw.py:137, in read(path_or_buffer, layer, encoding, columns, read_geometry, force_2d, skip_features, max_features, where, bbox, fids, sql, sql_dialect, return_fids, **kwargs)
    134 dataset_kwargs = _preprocess_options_key_value(kwargs) if kwargs else {}
    136 try:
--> 137     result = ogr_read(
    138         path,
    139         layer=layer,
    140         encoding=encoding,
    141         columns=columns,
    142         read_geometry=read_geometry,
    143         force_2d=force_2d,
    144         skip_features=skip_features,
    145         max_features=max_features or 0,
    146         where=where,
    147         bbox=bbox,
    148         fids=fids,
    149         sql=sql,
    150         sql_dialect=sql_dialect,
    151         return_fids=return_fids,
    152         dataset_kwargs=dataset_kwargs,
    153     )
    154 finally:
    155     if buffer is not None:

File /opt/conda/lib/python3.11/site-packages/pyogrio/_io.pyx:1046, in pyogrio._io.ogr_read()

File /opt/conda/lib/python3.11/site-packages/pyogrio/_io.pyx:745, in pyogrio._io.get_features()

FeatureError: GDAL returned more records than expected based on the count of records that may meet your combination of filters against this dataset.  Please open an issue on Github (https://github.com/geopandas/pyogrio/issues) to report encountering this error.

geopandas: 0.13.2
pyogrio: 0.6.0

Since the traceback so kindly asks to open an issue, I could not resist doing so... :smiley:

Thanks for reporting this!  I am able to reproduce the error locally.
One way to sidestep this is to use the use_arrow=True option on read_file.
The problem appears to be that GDAL is returning -1 for the number of features in each layer; ogrinfo also returns -1 for the feature count.
We use the feature count in various places to allocate arrays that we then populate when iterating over features, even when we tell it to give us the count the slow way (by iterating over all features) if the driver doesn't support a fast count (this driver does not).
The only idea I'm coming up with for this is that if we get back a -1 from GDAL, that we do our own loop over all records first to get the count, use that to allocate arrays, and then iterate over all the records again.  That seems pretty inefficient, but maybe is no different than what GDAL would be doing normally to get a count of records.

One way to sidestep this is to use the use_arrow=True option on read_file.

As far as I see this loads only Elements with Pointgeometries..

@brendan-ward I try to load this file:
import pyogrio
url = "http://download.geofabrik.de/europe/germany/baden-wuerttemberg-latest.osm.pbf"
pgdf = pyogrio.read_dataframe(url, use_arrow=True, layer="multipolygons")

but this returns an empty dataframe:
pgdf.info()

<class 'geopandas.geodataframe.GeoDataFrame'>
RangeIndex: 0 entries
Data columns (total 26 columns):
 #   Column       Non-Null Count  Dtype
---  ------       --------------  -----
 0   osm_id       0 non-null      object
 1   osm_way_id   0 non-null      object
 2   name         0 non-null      object
 3   type         0 non-null      object
 4   aeroway      0 non-null      object
 5   amenity      0 non-null      object
 6   admin_level  0 non-null      object
 7   barrier      0 non-null      object
 8   boundary     0 non-null      object
 9   building     0 non-null      object
 10  craft        0 non-null      object
 11  geological   0 non-null      object
 12  historic     0 non-null      object
 13  land_area    0 non-null      object
 14  landuse      0 non-null      object
 15  leisure      0 non-null      object
 16  man_made     0 non-null      object
 17  military     0 non-null      object
 18  natural      0 non-null      object
 19  office       0 non-null      object
 20  place        0 non-null      object
 21  shop         0 non-null      object
 22  sport        0 non-null      object
 23  tourism      0 non-null      object
 24  other_tags   0 non-null      object
 25  geometry     0 non-null      geometry
dtypes: geometry(1), object(25)
memory usage: 132.0+ bytes

Doing the same with url = "https://download.geofabrik.de/europe/liechtenstein-latest.osm.pbf" returns a filled dataframe.
Also when downsizing the file,
osmium tags-filter baden-wuerttemberg-latest.osm.pbf a/admin_level -o baden-wuerttemberg-admin.osm.pbf

the output returns a filled dataframe.
Is this related to the original issue?

Based on a lead found by @brendan-ward in #272 I gave the following snippet a try, and it seems to work, even though it crashes on my laptop without LIMIT clause because I don't have enough memory:
import pyogrio
url = "http://download.geofabrik.de/europe/germany/baden-wuerttemberg-latest.osm.pbf"
pgdf = pyogrio.read_dataframe(url, use_arrow=True, sql="SELECT * FROM multipolygons")

@CaptainInler I believe I have a fix for this now in #271; it uses a similar approach as ogr2ogr to set the layers to read from the file, which works for both use_arrow=True and without.  It works properly for the larger dataset you specify above as well as other large files I've tested.
However, because it has to do 2 passes over a lot of records the OSM file, not all of which are in the layer you read, it is slow to read from a remote URL.  I added some additional recommendations around this as part of #271, but in short, I highly recommend you download the file first.


Improve usage of httpc profile in integration and system tests

In scope of https://www.pivotaltracker.com/story/show/157656809
Please refer to commit message for details.

I cannot run system tests with this PR now, so keeping labelled as WIP.


Elm 2794 modularize glue jobs

@madhu-k-sr2 Terraform plan evaluation detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform - please contact #ask-modernisation-platform for assistance
Guidance on approving these PRs is available at https://user-guide.modernisation-platform.service.justice.gov.uk/runbooks/reviewing-mp-environments-prs.html#process-for-approving-prs

@madhu-k-sr2 Terraform plan evaluation detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform - please contact #ask-modernisation-platform for assistance
Guidance on approving these PRs is available at https://user-guide.modernisation-platform.service.justice.gov.uk/runbooks/reviewing-mp-environments-prs.html#process-for-approving-prs

@madhu-k-sr2 Terraform plan evaluation detected changes to resources that require approval from a member of @ministryofjustice/modernisation-platform - please contact #ask-modernisation-platform for assistance
Guidance on approving these PRs is available at https://user-guide.modernisation-platform.service.justice.gov.uk/runbooks/reviewing-mp-environments-prs.html#process-for-approving-prs


Latest version of VS Code breaks extensions

VSCode Version: 1.49.0
OS Version: macOS 10.15.6

with the latest release of vs code my extensions have broke. when trying to run them I get the error Error: Cannot find module 'vscode'. This even happens with a brand new extension generated with yo code

I have got PlatformIO break.
[platformio.platformio-ide]: Proposed API is only available when running out of dev or with the following command line switch: --enable-proposed-api platformio.platformio-ide

/duplicate #106499

/duplicate thanks @alefragnani

@alefragnani are you sure this is a duplicate? I'm not getting the same error and also not using the ActivityBar API


api v0 histories always return Transaction is irrelevant to given wallet

api histories description is Get the history of transactions.
return error:
Internal error occured: InternalError "Transaction is irrelevant to given wallet!"
the get url is
http://<IP_ADDRESS>/api/txs/histories?accountId=Ae2tdPwUPEZ18ZjTLnLVr9CEvUEUX4eW1LBHbxxxJgxdAYHrDeSCSbCxrvx@2147483648&address=DdzFFzCqrhtD4pm82QuSUp9SrocWfpnY6QFT5FmJjjtT6rqLoMUdXzQEg8w2VNXekYxf8GDmTSvA3vAKnx3UXJAgzg6tbQmd3gNmrof4&skip=0&limit=10

I can make sure accountId and address belong my wallet
if not send address params, still return Internal error occured: InternalError "Transaction is irrelevant to given wallet!" , why?

master brach return this error, but dev branch works fine:
{
    "Right": [
        [],
        0
    ]
}


[release-v1.4] Perform max.poll.interval.ms considering max.poll.records (#2439)

Cherry-pick of https://github.com/knative-sandbox/eventing-kafka-broker/commit/fe37776828e85c46524c7d82b5e8e54a789e9443

/assign @aliok @matzew

GitHub ...

+ local operator_dir=/tmp/serverless-operator
+ git clone --branch main https://github.com/openshift-knative/serverless-operator.git /tmp/serverless-operator
Cloning into '/tmp/serverless-operator'...
fatal: unable to access 'https://github.com/openshift-knative/serverless-operator.git/': The requested URL returned error: 429
+ export GOPATH=/tmp/go

/retest

/retest


Update index.html

Changes made:

Added the defer attribute to the script tag to ensure the JavaScript is executed after the HTML is fully parsed.
Corrected the title from "Bluring Image" to "Blurring Image" for proper spelling. This structure will ensure that the CSS and JavaScript are properly linked and executed in the correct order.

Hey @Imran-imtiaz48,
Thanks for your concern, Happy Coding....👍


Update .codecov.yml to ignore some pkg/trace utilities

What does this PR do?

Motivation

It's not useful to measure code coverage of test utility code, and is affecting the accuracy of our reporting tools
Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

Reviewer's Checklist

[x] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[x] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[x] A release note has been added or the changelog/no-changelog label has been applied.
[x] Changed code has automated tests for its functionality.
[x] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[x] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[x] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[x] If applicable, the need-change/operator and need-change/helm labels have been applied.
[x] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[x] If applicable, the config template has been updated.

So sorry for the noise, everyone. A bad rebase 🤦♀


Update some Ops to onnx v1.11.0

Use the latest opset for an Op in onnx v1.11.0 if there is no interface change
Handle the optional type which is new in the new onnx
Modify the opset warning based on the opset per op
Propagate the top opset of current onnx to Builder
Update the dialect document

Jenkins Linux ppc64le Build #5515 [push] Update some Ops to onnx ... started at 21:24

Jenkins Linux s390x Build #6393 [push] Update some Ops to onnx ... started at 21:22

Jenkins Linux amd64 Build #6377 [push] Update some Ops to onnx ... started at 20:22

Jenkins Linux amd64 Build #6377 [push] Update some Ops to onnx ... passed after 1 hr 13 min

Jenkins Linux s390x Build #6393 [push] Update some Ops to onnx ... passed after 1 hr 42 min

Jenkins Linux ppc64le Build #5515 [push] Update some Ops to onnx ... passed after 1 hr 59 min


Flag nodes as blocking

Allow flagging nodes as blocking and run them in core.async thread macro pool
Applicative Engine changes:

Fixed nil guarding for core-async
Added new protocol RunNode exclusively for running user supplied (leaf) functions
Added time comparison between lazy scheduler and applicative core-async (to our surprise, applicative seems to be faster)

Looks good, GitHub config isn't allowing me to review PR I opened.


feat: add error detection to connection sync and enable users to fix connections

Merging in orter to test sync conflicts on two separate devices


[Question]Longhorn workloads are not using the custom Priority Class

We have set the priority-class value to the name of a custom Priority Class. However, none of the Longhorn workloads are using it.
Longhorn Version
v1.0.2
PriorityClass
apiVersion: scheduling.k8s.io/v1
kind: PriorityClass
metadata:
  name: longhorn-node-critical
value: 1000000
globalDefault: false
description: "This priority class should be used for longhorn pods only."

Longhorn Settings
apiVersion: v1
kind: ConfigMap
metadata:
  name: longhorn-default-setting
  namespace: longhorn-system
data:
  default-setting.yaml: |-
    backup-target:
    backup-target-credential-secret:
    create-default-disk-labeled-nodes: true # Custom
    default-data-path:
    replica-soft-anti-affinity:
    storage-over-provisioning-percentage:
    storage-minimal-available-percentage:
    upgrade-checker:
    default-replica-count:
    guaranteed-engine-cpu:
    default-longhorn-static-storage-class:
    backupstore-poll-interval:
    taint-toleration: dedicated-execution:NoExecute; dedicated-scheduling:NoSchedule
    priority-class: longhorn-node-critical
    registry-secret:
    auto-salvage:
    disable-scheduling-on-cordoned-node:
    replica-zone-soft-anti-affinity:
    volume-attachment-recovery-policy:
    mkfs-ext4-parameters:

Longhorn UI

Hmm, this is weird. Do you see any error regarding the priority class in the longhorn-manager's log? Also, if you unset then set it, does it help?

FWIW, I just deployed 1.0.2 with the 1.0.201 helm chart and it appears the longhorn workloads do have the priorityClassName on the pods as expected.


Allow to disable frontend exception handler with None

@maximvlah

(Updated the tags, let me know if there is actually a bug and not just an improvement to be made)

@Lendemor it is actually a bug in rx.App._validate_exception_handlers. I will fix it tomorrow and open a PR.


Add BITBOX to projects section.

BITBOX is a suite of $BCH development tools. It includes a GUI and a command line utility.

Looks neat. =)


Enter to select Item from Result List

I would need to be able to use the enter key to trigger the selection event, if an element is selected, but also to submit the form if no item is selected. I use the resultlist as a list of "links", where I can currently use the tab key to trigger the event. The event is a simple "window.location.href". Hitting Enter just submits the form.
Basically a configurable key to trigger ("select") an event would be great. There might be something with the resultItem events, I might be missing, though. Perhaps it is possible to intercept the Enter key here and trigger a select-event instead of a form submission?

https://github.com/TarekRaafat/autoComplete.js/issues/189
https://github.com/TarekRaafat/autoComplete.js/issues/224
https://tarekraafat.github.io/autoComplete.js/#/configuration?id=submit-optional
events: {
	input: {
		keydown(event) {
			// Check pressed key
			switch (event.keyCode) {
				// Down/Up arrow
				case 40:
				case 38:
					event.preventDefault();
					// Move cursor based on pressed key
					event.keyCode === 40 ? autoCompleteInstance.next() : autoCompleteInstance.previous();
					break;
				// Enter
				case 13:
					if (!autoCompleteInstance.submit) event.preventDefault();
					// If cursor moved
					if (autoCompleteInstance.cursor >= 0) autoCompleteInstance.select(event);
					if (autoCompleteInstance.isOpen) {
						mysubmitfunc(autoCompleteInstance.feedback.selection)
					} else {
						mysubmitfunc()
					}
					break;
				// Tab
				case 9:
					// Select on Tab if enabled
					if (autoCompleteInstance.resultsList.tabSelect && autoCompleteInstance.cursor >= 0) autoCompleteInstance.select(event);
					break;
				// Esc
				case 27:
					// Clear "input" value
					autoCompleteInstance.input.value = "";
					autoCompleteInstance.close();
					break;
			}
		}
	}
}

First: Thank you very much! I figured out, that autoCompleteInstance is autoCompleteJS, aka the constant used in the examples to hold the configuration.
Second: I read those issues but couldn't make heads or tails out of them. But I hoped you would take a look at my issue and you didn't disappoint! Again, my gratitude.

Had some problems with the code above, but this one worked for me:
events: {         input: {             keydown(event) {                 switch (event.keyCode) {                     // Down/Up arrow                     case 40:                     case 38:                         event.preventDefault();                         event.keyCode === 40 ? autocomplete.next() : autocomplete.previous();                         break;                     // Enter                     case 13:                         if (autocomplete.cursor >= 0) {                             event.preventDefault();                             autocomplete.select(event);                         }                         break;                 }             }         }     }


Not a valid font file ttf error (using Windows release binary)

Describe the bug
Opening the generated ttf is giving an error.
.h file look fine.
That happened using one only font or/and merging/picking from many fonts.
Screenshots

Desktop (please complete the following information):
Windows 10
Additional context
Thats the binary that I used:
https://github.com/aiekick/ImGuiFontStudio/releases/download/b0.5/ImGuiFontStudio_Msvc_Win32.exe
Should I compile a new version from the repo by myself?
Thanks!

PS:
Trying to compile here.
I am very newbie out of Visual Studio...
Any help is appreciated.

I am trying to compile on the command prompt but I am getting this error:

PS D:\_CODE\_C\ImGuiFontStudio> cmake -B my_build_directory -DCMAKE_BUILD_TYPE=BuildMode
CMake Error: The current CMakeCache.txt directory D:/_CODE/_C/ImGuiFontStudio/my_build_directory/CMakeCache.txt is different than the directory /d/_CODE/_C/ImGuiFontStudio/my_build_directory where CMakeCache.txt was created. This may result in binaries being created in the wrong place. If you are not sure, reedit the CMakeCache.txt
CMake Error: The source "D:/_CODE/_C/ImGuiFontStudio/CMakeLists.txt" does not match the source "/d/_CODE/_C/ImGuiFontStudio/CMakeLists.txt" used to generate cache.  Re-run cmake with a different source directory.
PS D:\_CODE\_C\ImGuiFontStudio> cmake --build my_build_directory --config BuildMode
CMake Error: The current CMakeCache.txt directory D:/_CODE/_C/ImGuiFontStudio/my_build_directory/CMakeCache.txt is different than the directory /d/_CODE/_C/ImGuiFontStudio/my_build_directory where CMakeCache.txt was created. This may result in binaries being created in the wrong place. If you are not sure, reedit the CMakeCache.txt
The system cannot find the file specified
CMake Error: Generator: execution of make failed. Make command was: /usr/bin/make.exe -f Makefile &&
PS D:\_CODE\_C\ImGuiFontStudio>

I also tried on MSYS1 32/64 but I am having errors... OpenGL looks that is installed...

moebiussurfing@surfingMachine MINGW64 /d/_CODE/_C/ImGuiFontStudio
$ pacman -Ss glew
mingw32/mingw-w64-i686-glew 2.2.0-2 [installed]
    GLEW, The OpenGL Extension Wrangler Library (mingw-w64)
mingw64/mingw-w64-x86_64-glew 2.2.0-2 [installed]
    GLEW, The OpenGL Extension Wrangler Library (mingw-w64)
ucrt64/mingw-w64-ucrt-x86_64-glew 2.2.0-2
    GLEW, The OpenGL Extension Wrangler Library (mingw-w64)
clang32/mingw-w64-clang-i686-glew 2.2.0-2
    GLEW, The OpenGL Extension Wrangler Library (mingw-w64)
clang64/mingw-w64-clang-x86_64-glew 2.2.0-2
    GLEW, The OpenGL Extension Wrangler Library (mingw-w64)

moebiussurfing@surfingMachine MINGW64 /d/_CODE/_C/ImGuiFontStudio
$ cmake -B my_build_directory -DCMAKE_BUILD_TYPE=BuildMode
System is unknown to cmake, create:
Platform/MINGW64_NT-10.0-19044 to use this system, please post your config file on discourse.cmake.org so it can be added to cmake
Your CMakeCache.txt file was copied to CopyOfCMakeCache.txt. Please post that file on discourse.cmake.org.
CMake Error at /usr/share/cmake-3.22.1/Modules/FindPackageHandleStandardArgs.cmake:230 (message):
  Could NOT find OpenGL (missing: OPENGL_opengl_LIBRARY OPENGL_glx_LIBRARY)
Call Stack (most recent call first):
  /usr/share/cmake-3.22.1/Modules/FindPackageHandleStandardArgs.cmake:594 (_FPHSA_FAILURE_MESSAGE)
  /usr/share/cmake-3.22.1/Modules/FindOpenGL.cmake:443 (FIND_PACKAGE_HANDLE_STANDARD_ARGS)
  cmake/3rdparty.cmake:14 (find_package)
  CMakeLists.txt:22 (include)

-- Configuring incomplete, errors occurred!
See also "/d/_CODE/_C/ImGuiFontStudio/my_build_directory/CMakeFiles/CMakeOutput.log".

I compiled the current master branch binary,
but still getting the same error...
Any idea?

Hello,
i know there is some problems with some fonts
chek this past issue : #14
the goal of this tool was first for programming and especially for imgui.
there is better tool for general use, like fontforge.
Not sure why its broken, the generation of font is partial, by ex, i not support composites glyphs and some others features.
this is why there is no release for the moment.
btw you have the code, so you can try to found the bug, because i have no time to work on it for many months.

Ok. thanks.
I tried to open the "broken" ttf and the .h into my ImGui project codes and it worked.
I don't need FontForge at all, or to open the font anywhere else,
I just want to make my ImGui buttons.

Now I need to learn a bit to layout fine into the buttons, centered etc..

i think its better to use two font instead of one merged.
because when you merge glyphs, you have source fonts who have differents scale, base line and other things, than can produce different placement of glyphs in the same font. thais is why in ImGuiFontStudio there is a pane for preview the final font for a text.

if you merged two font who are using the same params, its ok but if not can be an issue.
for your issue, you need to check the font scale at start


evaluate if switching to deno would be a good idea

<EMAIL_ADDRESS>

Deno is a bit overkill and discord.js doesn't support it yet, maybe bring this issue back up in 6 months - a year if you feel super strongly about deno


apply new switch capability to disable integer overflow checks

and make various other refinements in text

On 30-Sep-20 3:01 PM, Gustavo A. Hoffmann wrote:

@gusthoff approved this pull request.
It looks very good!

Thanks!


Feature/setup

What does this PR do?
This PR is raised to implement the following features 👍:

[x] Build a To-Do app using React and Vite
[x] The app adds, removes,s and edits new tasks
[x] Use react icons
[x] Save tasks in local storage
[x] Use different hooks to improve performance

Description of the completed tasks 🙌

[x] Added the HTML for the structure.
[x] Added the CSS file for the styling.
[x] Added the js components for React.
[x] Used the linter tool that analyzes my source code to flag programming errors, bugs, stylistic errors, and suspicious constructs. 💯

Thank you @Mar12358 for the thorough review. By mistake I started this pull request to the wrong branch 'main' when I should make it to the 'development' branch. I will address the suggested changes in another pull request. Please review my new Pull Request

Thank you @Mar12358 for the thorough review 🥳


React Redux Group Capstone

👇 👇 The following are the features added to the repo 👇 👇

Our Kanban Board Link

We are two in our group.

Hi @Peter-Dumbari ,
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
Highlights

Board setup✔
Task assignment✔

Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


New schema locater to locate the standard schemas delivered as part of the core-backend package

Fixes https://github.com/iTwin/itwinjs-core/issues/6145

I don't think this is the right solution to the problem.
This change adds three locaters.  Only one is necessary.  A schema context takes multiple locaters so if a user wants to add paths they can add additional locaters to the context.
This change also combines ability to load from packages installed in node_modules with the requested ability to load schemas deployed with the backend.  Loading assuming node_modules is fixed has limited use as things may be packaged in a real deployment.
I suggest implementing one locater that takes the backend assets dir string (must be defined) and internally adds paths for the sub directories needed.  It should not accept any other paths from the user.  It should be explicitly documented so the user knows how to construct it and what schemas it will locate.
Locaters added to a context are checked in the order they are added, so the last locater added is the last one checked.  So it is possible to construct a schema context with this locater as the lowest precedence locater but that is easy to mess up.  One possible strategy to improve this would be to follow the pattern from imodel_native where you can add a last chance locater.
It would be better if it 'just worked' without having to take a path or requiring the user to register this locater correctly but that is not possible because core-backend does not have a dependency on any of the ecschema packages.

Okay sounds good ! I will strip down the locater to focus only on the backend schemas.
I was thinking of this as a possible replacement/extension of the existing locater with the added functionality of being able to locater backend schemas.
I added the node-modules search code to be able to load in a wider set of schemas including their json versions as the backend only puts out xml files.


Zypper package resource doesn't respect options

Description
For example, using:
package 'rpm-build' do
  options '--force-resolution'
end

will end up with the invocation of:
zypper --non-interactive --no-gpg-checks install --auto-agree-with-licenses rpm-build=4.11.2-15.1
rather than the expected:
zypper --non-interactive --no-gpg-checks install --auto-agree-with-licenses --force-resolution rpm-build=4.11.2-15.1
Chef Version
12.9.38 +
Platform Version
sles 12.1 (EC2 ami to be specific)
Replication Case
Tell us what steps to take to replicate your problem.  See How to create a Minimal, Complete, and Verifiable example
for information on how to create a good replication case.
Client Output
The relevant output of the chef-client run or a link to a gist of the entire run, if there is one.
The debug output (chef-client -l debug) may be useful, but please link to a gist, or truncate it.
Stacktrace
Please include the stacktrace.out output or link to a gist of it, if there is one.
NOTE: CHEF CLIENT BUGS ONLY
This issue tracker is for the code contained within this repo -- chef-client, base knife functionality (not
plugins), chef-apply, chef-solo, chef-client -z, etc.

Requests for new or alternative functionality should be made to feedback.chef.io
Chef Server issues
ChefDK issues
Cookbook Issues (see the https://github.com/chef-cookbooks repos or search Supermarket or GitHub/Google)

yeah both the shell_out calls need new_resource.options added to it, not sure if i took that out or if its simply never supported it.
should consider if the options flag should overwrite stuff like --no-gpg-check or not.


Allow configuration of "Event sources"

This plugin is great for development workflow and quickly uploading code and configurations to Lambda.
The one exception is "Event Sources".  Currently this is something we have to manually set on each upload - in our case, it's the "Alexa Skills Kit".
Is this something that could be configured via lambduh?
cc @philipmw

Sorry, I've been away on travel, I'll look into this soon.

I'm going to start working on this shortly.

@SeanRoy Just curious - any ETA for this?

@barbeau from version 2 plugin has support for SNS "topics" and scheduled rules "scheduledRules", please refer to https://github.com/SeanRoy/lambda-maven-plugin/blob/master/README.md

Hi Sean, thanks for your email.  I'm going to try to get this in by the end
of October.
<EMAIL_ADDRESS><EMAIL_ADDRESS>| nrby.com
Twitter https://twitter.com/nrbynow  |  Facebook
https://www.facebook.com/nrbynow  |  Instagram
https://instagram.com/nrbynow/  |  LinkedIn
http://linkedin.com/company/nrby
http://linkedin.com/company/nrby
This message is intended only for the use of the individual or entity named
above and may contain information which is privileged, confidential, and
not intended for disclosure. If you are not the intended recipient, you are
hereby notified that any disclosure, copying, distribution, dissemination
or use of this message is strictly prohibited. If you have received this
message in error, please accept our apology. We would very much appreciate
your notifying the sender immediately by return email. Thank you.
On Sat, Oct 8, 2016 at 1:00 PM, Krzysztof Grodzicki <
<EMAIL_ADDRESS>wrote:

@barbeau https://github.com/barbeau from version 2 plugin has support
for SNS "topics" and scheduled rules "scheduledRules", please refer to
https://github.com/SeanRoy/lambda-maven-plugin/blob/master/README.md
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/SeanRoy/lambda-maven-plugin/issues/18#issuecomment-252435871,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIycDuRURkGL9sMXqDDhpgJY9Gi5I-QUks5qx8wygaJpZM4HbxHT
.

I've begun work on this.

@barbeau I just deployed the alexa skills kit integration to 2.1.3.  Please note that this change just allows the lambda function to be triggered by alexa.  You'll still have to do the manual steps of putting the function arn in Alexa and the Alexa app id into the lambda function.  At this time I'm not sure there's a way to automate that.
I'm still waiting for my amazon echo to do a true end to end test of this functionality.

Awesome,  thanks,  I'll try this out!

BTW,  here are some ways to test without physical Alexa device:
https://github.com/OneBusAway/onebusaway-alexa/blob/master/README.md#testing-without-an-alexa-device
Roger was the closest I could get,  although it's not perfect.

How is this working out for you?  Can I close this issue?

Sorry, haven't had a chance to try it yet - I've been having certification
issues with Amazon so I've been stuck in that state.  Feel free to close
and I'll reopen if it doesn't work when I try it.  Thanks for implementing
this!
On Nov 10, 2016 8:46 PM, "Sean Roy"<EMAIL_ADDRESS>wrote:

How is this working out for you? Can I close this issue?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/SeanRoy/lambda-maven-plugin/issues/18#issuecomment-259859949,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA4pLe1CaaHq_nAYmxSvy5GUNjgB4NpXks5q88jjgaJpZM4HbxHT
.


Failed to perform, ErrCode: 27, Reason: ''

Im trying to do the same thing as i did before with pycurl on linux, but here im stuck with some weird problem:
Script closes on second perform with Failed to perform, ErrCode: 27, Reason: '' , and i cant seem to be able to send second request inside one session.
In another cofings i tried -  they crashed without even throwing an error but and still couldnt execute second perform.
What i have found - error 27 is a buffer overflow, but my example has pretty low data usage and there should be no overflow in here. point me where i might be wrong?
from io import BytesIO
from curl_cffi import Curl, CurlOpt, CurlInfo

def test():
    ua = "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36"
    custom_headers = [
        "referer:https://google.com/",
        f"user-agent:{ua}",
        ]

    ch = Curl()
    buffer = BytesIO()
    ch.impersonate("chrome99")

    ch.setopt(CurlOpt.USERAGENT, bytes(ua, 'utf-8'))
    ch.setopt(CurlOpt.URL, 'https://ipv4.icanhazip.com/')
    ch.setopt(CurlOpt.WRITEDATA, buffer)
    ch.setopt(CurlOpt.HTTPHEADER, [bytes(a, 'utf-8') for a in custom_headers])
    ch.perform()
    body = buffer.getvalue()
    print(body)
    print("================================")

    ch.setopt(CurlOpt.URL, 'https://ipv4.icanhazip.com/')
    ch.setopt(CurlOpt.WRITEDATA, buffer)
    ch.setopt(CurlOpt.HTTPHEADER, [bytes(b, 'utf-8') for b in custom_headers])
    ch.perform()
    body = buffer.getvalue()

    print(body)

    print('ENDS HERE')
    ch.close()

test()

Second error type when i comment most of setopt's before second perform

def test():
    ua = "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36"
    custom_headers = [
        "referer:https://google.com/",
        f'user-agent:{ua}',
        ]

    ch = Curl()
    buffer = BytesIO()
    ch.impersonate("chrome99")

    ch.setopt(CurlOpt.USERAGENT, bytes(ua, 'utf-8'))
    ch.setopt(CurlOpt.URL, 'https://ipv4.icanhazip.com/')
    ch.setopt(CurlOpt.WRITEDATA, buffer)
    ch.setopt(CurlOpt.HTTPHEADER, [bytes(a, 'utf-8') for a in custom_headers])
    ch.perform()
    body = buffer.getvalue()
    print(body)
    print("================================")

    #buffer = BytesIO()
    #ch.setopt(CurlOpt.URL, 'https://ipv4.icanhazip.com/')
    #ch.setopt(CurlOpt.WRITEDATA, buffer)
    #ch.setopt(CurlOpt.HTTPHEADER, [bytes(b, 'utf-8') for b in custom_headers])
    ch.perform()
    body = buffer.getvalue()

    print(body)

    print('ENDS HERE')
    ch.close() ```

Sorry, this is a serious bug. The cause is that I'm not reseting the header pointer to ffi.NULL in curl.perform, I will fix this soon in the next minor version.

Please update by pip install -U curl_cffi, the lastest version(0.3.7) should have this issue fixed.

Oops

Windows wheel still not updated as mentioned in #13 , I'll fix that ASAP.

If you are in a hurry, just add one line in your site-packages/curl_cffi/__init__.py like here.

If you are in a hurry, just add one line in your site-packages/curl_cffi/__init__.py like here.

Yeap, already, but all the same, error still there

build error or perform error?

build error or perform error?

My mistake, now all fine, problem solved, thank you!


Title: Resistration page need improvement

Is your feature request related to a problem? Please describe.
##Description
The registration page of the website needs improvement the UI of the registration page is very unattractive which will effecting the user experience. I want to improve the Registration page UI
Describe the solution you'd like.
Want to improve the UI of the registration page by adding some additional features in the registration page
Also the registration page styling need improvement
Describe alternatives you've considered.
The user-friendly UI is needed on the registration page
Add any other context or screenshots about the feature request here.
##Screenshot

Sure @HimanshuNarware

I would like to work on this project . Please assign it to me.


Can't open connection for MySqlConnection()

I using Dapper instead EFCore and it's very faster and then I convert to LambdaNative and its have a problem.
I debugging line by line and I found some method/properties of MySqlConnection can't use such as

_connection.State
_connection.Open();

and I got error message
START RequestId: 45396de3-dda5-4ab9-9774-65d0b4a9a22b Version: $LATEST
END RequestId: 45396de3-dda5-4ab9-9774-65d0b4a9a22b
REPORT RequestId: 45396de3-dda5-4ab9-9774-65d0b4a9a22b	Init Duration: 56.35 ms	Duration: 6901.78 ms	Billed Duration: 7000 ms 	Memory Size: 1024 MB	Max Memory Used: 427 MB
RequestId: 45396de3-dda5-4ab9-9774-65d0b4a9a22b Error: Runtime exited with error: signal: segmentation fault (core dumped)
Runtime.ExitError

I don't know why
This my code
https://github.com/ifew/aws-lambda-lambdanative-db-dapper/blob/master/src/aws-lambda-lambdanative-db-dapper/Function.cs

Perhaps try compiling a simple console app that uses Dapper with CoreRT. It's quite possible that it uses a something that isn't supported by CoreRT yet. I don't think it would be LambdaNative causing this problem.
If you find it doesn't work under CoreRT, you could raise an issue with them. They're usually quite helpful.

Perhaps try compiling a simple console app that uses Dapper with CoreRT. It's quite possible that it uses a something that isn't supported by CoreRT yet. I don't think it would be LambdaNative causing this problem.
If you find it doesn't work under CoreRT, you could raise an issue with them. They're usually quite helpful.

OK, I develop example ifew/corert-db-dapper and it's work.

Your example doesn't have the Microsoft.DotNet.ILCompiler package so it can't be building with CoreRT. It looks like it will just be building with .NET Core

@zaccharles OK, after I added Microsoft.DotNet.ILCompiler its fail, I think from reflection. and  I will ask from Corert Issues Board
but you have any ideas for getting a list of runtime directive?
Thank you

OK, I found the root cause from CoreRT has not to support some ORM. and now I have completed with ADO.NET at https://github.com/ifew/aws-lambda-lambdanative-db
Thank you for support


Could I get the training code for DCVC-DC

Hi, thank you for sharing the repository.
I would like to fine-tune DCVC-DC to adapt to my dataset.
Could I receive the training code for this purpose?
Please contact me at<EMAIL_ADDRESS>Thank you.

I'm so interested in the DCVC-FM, could you please send me the traing codes. Thanks very much.
<EMAIL_ADDRESS>

I'm so interested in the DCVC-FM, could you please send me the traing codes. Thanks very much.
<EMAIL_ADDRESS>


refactor(ES6): upgrade WebWorkerHotUpdateChunkTemplatePlugin to ES6

What kind of change does this PR introduce?
refactor, ES5 => ES6

Did you add tests for your changes?
All existing tests pass

If relevant, link to documentation update:
N/A

Summary
upgrade WebWorkerHotUpdateChunkTemplatePlugin to ES6

Does this PR introduce a breaking change?
No

Thanks


Examples throw FnMut error

The following error is thrown when trying to run examples;
expected a 'FnMut(&mut Ui)' closure, found 'GraphView<'_, (), ()>
On lines in the update function where we ui.add(&mut GraphView::<...

what example have you tried to run?

Figured out my issue - error was behind the keyboard - I hadn't properly copied some of the "use" statements, so I was using petgraph::graph::Graph instead of egui_graphs::Graph


Large objects were  Free but space is not compressed

Using this version of ASP.NET Core '2.2.6'
Docker image from mcr.microsoft.com/dotnet/core/aspnet:2.2.6-stretch-slim
Centos7

I disable servergarbage , but the monory useage is still growing 。
this is objs large than 85000
> dumpheap -min 85000 -live
         Address               MT     Size
00007eff203d1450<PHONE_NUMBER>416bc0   862502 Free
00007eff2056ced8<PHONE_NUMBER>416bc0  6825246 Free
00007eff20c2e280<PHONE_NUMBER>416bc0   795766 Free
00007eff20d16388<PHONE_NUMBER>416bc0  2578806 Free
00007eff20ffe1e0<PHONE_NUMBER>416bc0 11218598 Free
00007eff21e0f228<PHONE_NUMBER>416bc0   617774 Free
00007eff21fe7e58<PHONE_NUMBER>416bc0  2072062 Free
00007eff221fa380<PHONE_NUMBER>416bc0 12560726 Free
00007eff22e75860<PHONE_NUMBER>416bc0  1883350 Free
00007eff233f4148<PHONE_NUMBER>416bc0  6165478 Free
00007eff239d5590<PHONE_NUMBER>416bc0  4724478 Free
00007efe2f00c670<PHONE_NUMBER>416bc0   783614 Free
00007efe2f28ba38<PHONE_NUMBER>416bc0  2153862 Free
00007efe2f7228b8<PHONE_NUMBER>416bc0  1077822 Free
00007efe2fe3e1e0<PHONE_NUMBER>416bc0  2113414 Free
00007efe301aed50<PHONE_NUMBER>416bc0   612246 Free
00007eff24157460 00007eff3f30d9b8  1146880
00007eff24273478 00007eff341e50f0  1024024
00007eff2436d4b0 00007eff3f6075c8  1146880
00007eff249a9418 00007eff341e50f0  1024024
00007eff24aa3430<PHONE_NUMBER>416bc0 12671046 Free
00007eff257d7178<PHONE_NUMBER>416bc0  3326966 Free
00007eff25b57798<PHONE_NUMBER>416bc0  3421350 Free
00007eff2603aef8<PHONE_NUMBER>416bc0 41828486 Free
00007eff2891f120<PHONE_NUMBER>416bc0   700046 Free
00007eff28b6a268<PHONE_NUMBER>416bc0  3327918 Free
00007eff28eb6a30<PHONE_NUMBER>416bc0  1667046 Free
00007eff29157570 00007eff341e50f0  1024024
00007eff2928a608 00007eff341e50f0  1024024
00007eff29384620<PHONE_NUMBER>416bc0 24382670 Free
00007eff2ad449a0<PHONE_NUMBER>416bc0 18350502 Free
00007efe3ffff000<PHONE_NUMBER>416bc0 58592070 Free
00007efe4385f218<PHONE_NUMBER>416bc0 30602614 Free
00007efe45901a08<PHONE_NUMBER>416bc0 33554574 Free
00007efe47961b20<PHONE_NUMBER>416bc0  5910718 Free
00007efda3fff000<PHONE_NUMBER>416bc0 133333294 Free
00007efd93ffb000<PHONE_NUMBER>416bc0 133440718 Free
00007efd8bff9000<PHONE_NUMBER>416bc0 132632214 Free
00007efd83ff7000<PHONE_NUMBER>416bc0 133538390 Free
00007efd7bff5000<PHONE_NUMBER>416bc0 49981550 Free

Statistics:
              MT    Count    TotalSize Class Name
00007eff3f6075c8        1      1146880 UNKNOWN
00007eff3f30d9b8        1      1146880 UNKNOWN
00007eff341e50f0        4      4096096 System.Byte[]
0000000001416bc0       34    878307916      Free
Total 40 objects

What I want to know is why the free   memory is not compressed.
I use gcwhere command  and the obj gen is 3
> gcwhere 00007eff24273478
Address            Gen   Heap   segment            begin              allocated           size
00007EFF24273478   3      0     00007EFF23FFE000   00007EFF23FFF000   00007EFF2BFE4D20    0xfa018(1024024)

It means this obj is in large object heap?  I only know that there are 0,1,2 gen

You are over allocating on the LOH. Fragmentation is a consequence of this. I'd recommend looking at your allocation strategy by either pooling memory usage or by breaking up larger allocations to stay within the 85k limit way. The GC provides ongoing optimizations on how the LOH is maintained and I'm sure there will be further improvements in this area.

You are over allocating on the LOH. Fragmentation is a consequence of this. I'd recommend looking at your allocation strategy by either pooling memory usage or by breaking up larger allocations to stay within the 85k limit. The GC provides ongoing optimizations on how the LOH is maintained and I'm sure there will be further improvements in this area.

Can Free memory in LOH be compressed by setting  GCLOHCompact = CompactOnce ?

I'm not sure I could reccomend forcing a collection. Are you actually experiencing memory issues or is this more of a diagnostics task? Pooling the byte arrays may reduce gc pressure but I would recommend looking at stats, the fragmentation maybe small enough to not be an issue.


Day 8

Hello!
The day_8 folder is missing :)

do you read book or notion template? this lesson does not contain code examples


Add Trace Sample

Adds a working sample for spring-cloud-gcp-trace.
Traces are correctly sent to Cloud Trace console.
Still wasn't able to figure out how to get log correlation working and how to get it to shutdown cleanly. I have a feeling that this is due to spring-graalvm-native missing some functionality but not completely sure.

Since logging works separately, what's happening here? Are the log statements making it over to Stackdriver at all, but without the trace id metadata?

Yes, so the Logging library works (as it is just a GAPIC generated library), however the Logback framework does not work yet. So that is why this is not working.


[Fix-10665] [S3] Fix s3 download method

Purpose of the pull request

fix s3 download error, which cause task failure
fix #10665
Brief change log

when overwrite param is true and dest file is exist, throw exception
make dirs for ancestor directories of the source file path if it isn't exists, if create failed, throw exception
To describe the error more clearly, change "the file isn`t exists" to specific error message

Verify this pull request

This pull request is code cleanup without any test coverage.

Hi @sketchmind , to target specific issue from PR, we recommend use keyword fix: #issue_id or close: #issue_id or closes: #issue_id in your PR describe(not title, just desc). It would not only connect issue to PR but also close issue automatically when PR is be closed.

Hi @sketchmind , to target specific issue from PR, we recommend use keyword fix: #issue_id or close: #issue_id or closes: #issue_id in your PR describe(not title, just desc). It would not only connect issue to PR but also close issue automatically when PR is be closed.
Please pause the code merging, I need to revise it

I recreate the branch Fix-10665，how to reopen it

I recreate the branch Fix-10665，how to reopen it

The PR of the deleted branch cannot be reopened. You can create a new one.


Add DDPG and fix Gym wrapper

Could you run rustfmt? Seems to break in the CI.

Done! I have to leave my laptop now, hopeful everything is good, otherwise tomorrow :)

Merged after pushing some small cleanups, thanks!


Replaced Echo dependency from the core integration with a hook

As far as I can tell this is the minimal set of changes required for the extension to work, and let us override notification. There are more Echo references, and echo-specific hook handlers which I've decided to leave alone, as they seem to be dead code in our context, and removing them would introduce too much change from the original extension to remain easy to upgrade

Have you tried running Phan on the extension's code? I was wondering if we could switch it with possibly very little effort.

Have you tried running Phan on the extension's code? I was wondering if we could switch it on with possibly very little effort.

Yep I have, and it has time with all the Echo referenced in dead parts of extension. We would need to disable a lot of the checks or go and change a bunch of extension code.
Additionally, I think that we were wrong to even put that into ACs in the first place. It's not a Gamepedia extension that we treat like third party, even though it's technically owned by us. In this case we probably want to keep it as close to third party original as we can, and trying to run our PR checks on this doesn't give us much value

Let's try to get Phan running. If we checkout Echo as well, I believe it should discover its classes and work.

The branch is currently checking out Echo and Flow, but I don't think the classes are being loaded


Global Typescript definition

The global typescript definition is an amazing addition, thank you! It has already been added to https://github.com/patmood/pocketbase-typegen
I have a suggestion to improve the return type:
Here's the example from the docs
interface Task {
  id:   string;
  name: string;
}

interface TypedPocketBase extends PocketBase {
  collection(idOrName: 'tasks'): RecordService<Task>
}

const pb = new PocketBase("http://<IP_ADDRESS>:8090") as TypedPocketBase;

pb.collection('tasks').getOne("RECORD_ID") // -> results in Promise<Task>

I think the results should automatically extend RecordModel instead of just Task, so it includes fields like id, expand, created,updated etc
The reason the user can't manually do this in the TypedPocketBase interface is because that basic Task type is required for create/update methods.
Is this able to be inferred by the sdk?

I'm sorry, I'm not sure I follow what actually is the issue here.
If users want to extend RecordModel then they can do that on their own, it is not much different then the explicit generic getOne<Task>().

When using global type definition create() expects the same Task type that is returned by getOne, but they are not the same.
getOne returns extra fields like id/collectionId/collectionName/created/updated/expand.
This isn't an issue with explicit generics because you can specify different types for each method (pocketbase-typegen generates a Task and a TaskResponse to handle this).

Task just describes a subset of common fields the user want to access. It doesn't perform cast or anything like that. If users want TS to not complain when accessing collectionId, collectionName, etc. then they can define it in the interface (or extends it on their own via the RecordModel). This is exactly the same as the individual methods generic, there is really no difference. The global generic just sets a default type for the generic arguments (previously the default was hardcoded to RecordModel).
Also note that getOne, getList, create, etc. methods all can return custom subset of fields when the fields parameter is set.

create() expects the same Task type that is returned by getOne

I don't understand this. I haven't personally used [pocketbase-typegen] so I'm not sure if this is something specific to it, but the accepted body argument for the create method in the JS SDK should be something like: {[key:string]:any}|FormData because technically it can be anything (including custom data that is submitted only to be checked in the API rules via @request.data.*).
In any case I don't think this is an issue with the SDK, or at least I don't understand the root of the problem. If you think that something can be changed to make it easier integrating with your package, feel free to open a PR with the suggestion and I'll have a look.


[20.09] apacheHttpdPackages.mod_auth_mellon: 0.13.1 -> 0.17.0

(cherry picked from commit 90649bacfbc4fd5a8d77f23135d34bc69ca10229)

Motivation for this change
Backport of https://github.com/NixOS/nixpkgs/pull/115135, closes https://github.com/NixOS/nixpkgs/issues/93102.
Things done

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

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

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

@GrahamcOfBorg build apacheHttpdPackages.mod_auth_mellon


Updated AMI ID

Hope I picked the right one.

It might be a better idea to just ask people to search for rubber-docker under public amis


ftp: README review

subtask from:
https://github.com/pombase/pombase-chado/issues/720
https://github.com/pombase/website/issues/2058

@kimrutherford  add file names of new structure to this document
https://docs.google.com/document/d/1TfvWngsI2U9-wkw2czxhHOZNmmQ8nxRwYa-TctrhMs0/edit

@PCarme  write READMEs (a lot of the info will be on the downlades website, do decide how much detail required and add referring URLs. If you don't know what it is tag me in the doc, I will add

Val/Lim to review

Pascal to cipy into place in Git

add file names of new structure to this document

I can do that but does it make sense to duplicate what is already in Git?  Can't we edit the README text directly rather than copy into a Google doc and then back to the files in Git?
The READMEs are here:
https://github.com/pombase/pombase-scripts/tree/main/release_readme_files
There is one README file for each of the directories in the new structure:
https://www.pombase.org/public_releases/pombase-2024-06-01/

good point!

The READMEs are here:
https://github.com/pombase/pombase-scripts/tree/main/release_readme_files

Okay, thanks Kim ! I'll review the READMEs in there, and let you know when I'm done.

In https://github.com/pombase/pombase-scripts/blob/main/release_readme_files/exports_for_external_resources-README.txt, I have listed the files in the directory, but I don't really know what each of those corresponds to.

I have listed the files in the directory, but I don't really know what each of those corresponds to.

Thanks Pascal.  I'll work on that one.

The "genome_sequence_and_features" directory contains several subdirectory. Should there be READMEs for all subdirectory, or a single README describing the content of all the subdirectories ?

Also, this file https://www.pombase.org/public_releases/pombase-2024-06-01/protein_features/transmembrane_domain_coords_and_seqs.tsv displays the entire sequence of each protein, not just the transmembrane domains sequences. Is it intended like that ?

It says coordinates and sequences, but it seems strange to put them together...
Maybe this wasn't a file for the public?
@kimrutherford ?

It says coordinates and sequences, but it seems strange to put them together...
Maybe this wasn't a file for the public?

This is all I can find about it:

https://github.com/pombase/website/issues/1215

This is all I can find about it:

I dug into my old email.  This is from Snezhka.  The thread is from April 2019, with the subject "transmembrane domains":

Hope everything is well - writing now to bug you with a question, sorry... Wonder if there is a way to, say, 'automatically' collect all transmembrane domains from all proteins. What I want to do is to compare the transmembrane domains (e.g. length distribution, unusual amino acids) in S. pombe to those in S. japonicus. Ideally so that I could do it separately for single spanners vs multispanners.

The file was created for Snezhka but it's updated nightly.  Perhaps we don't need it in the new release directories?

Perhaps we don't need it in the new release directories?

agree, it's a bit random

agree, it's a bit random

OK, I've removed that file from the script that creates the new release directory structure.

The contents are quite diverse so I think each directory needs a README

I've added empty READMEs and checked that the script can process README files for sub-directories correctly.

Why is this file https://www.pombase.org/public_releases/pombase-2024-06-01/genome_sequence_and_features/fasta_format/feature_sequences/introns_within_cds.fa not just called introns.fa ?

Also, this file isn't loaded properly https://www.pombase.org/public_releases/pombase-2024-06-01/genome_sequence_and_features/gff_format/Schizosaccharomyces_pombe_all_chromosomes_unstranded.gff3

I had the same issue
https://github.com/pombase/website/issues/2219
but Kim has not been able to replicate

different issue!

There is a file with introns in CDS only (more important  that we have these annotated), and one with CDS+UTRs
(we started adding UTR introns later, and we definitely don't have them all)

Oh right ! I hadn't thought about the UTR introns, it makes sense then. Thanks !

Also, this file isn't loaded properly https://www.pombase.org/public_releases/pombase-2024-06-01/genome_sequence_and_features/gff_format/Schizosaccharomyces_pombe_all_chromosomes_unstranded.gff3

I think that's OK.  The file is empty because we don't have any unstranded features.  Maybe we did have some years ago.  I think it's best to remove it to prevent confusion.

I'm done writing the READMEs by the way.

I'm done writing the READMEs by the way.

Excellent.  Thanks!
I haven't completed exports_for_external_resources-README.txt yet.  Once I have, I'll make an example releases directory for 2024-09-01 so we can see if there is anything else needed.

Here's how the structure looks with the new READMEs and the latest release:
https://www.pombase.org/public_releases/pombase-2024-09-01/
We currently have the GPI/GPAD files for GO in this directory:
https://www.pombase.org/public_releases/pombase-2024-09-01/exports_for_external_resources/
Maybe they should be in the gene_ontology directory?  It could be a sub-directory.

I've moved the allele_summaries.json file from exports_for_external_resources to the training_data_for_ML_and_AI directory since that's what is was created for (I think).  There's nothing stopping us having files in more than one place so we could have a copy in exports_for_external_resources if it makes sense.

I haven't completed exports_for_external_resources-README.txt yet.

I've done that now:
https://www.pombase.org/public_releases/pombase-2024-09-01/exports_for_external_resources/PomBase_exports_for_external_resources_README.txt
As an experiment, the format is a bit different from the other READMEs.  Let me know if you think it's better or worse.

Once I have, I'll make an example releases directory for 2024-09-01 so we can see if there is anything else needed.

I've done that too.  Perhaps we can have a chat about it once we're all back from holiday.
https://www.pombase.org/public_releases/pombase-2024-09-01

I agree it makes sense to have the official GO release in the GO directory

I've moved the GPI/GPAD files into the gene_ontology directory.
https://www.pombase.org/public_releases/pombase-2024-09-01/


[dataplane] reset ActiveStream::streamInfo().route() when clearRouteCache()

Title: reset ActiveStream::streamInfo().route() when clearRouteCache()
Description:
There is a gap between the ActiveStreamFilterBase::clearRouteCache and  ActiveStreamFilterBase::streamInfo()::route():
whenever we call clearRouteCache(), the  {en,de}coder_callback_.route() ONLY!, the streamInfo().route() is NOT updated.
The "new" route() on {en,de}coder_callback_ will be reset when we refreshCachedRoute(), by either explicitly calling it, or calling {en,de}coder_callback_.route().
So after a clearCachedRoute(), the route() on streamInfo is "stale" and will be out-of-date until refreshCachedRoute() is called.
we should explicitly clear/reset streamINfo()::route_ in clearRouteCache()

@yanavlasov

ping

can we nostale-bot this issue please.

/nostalebot


Allow trailing required positional parameters

This issue was originally filed by @seaneagan

Problem:  It's much more readable to pass callbacks as the last argument to a function call so as not to bury other args behind a long code block.  But that's not currently possible if:

the callback is required
the function call also has optional args

since optional args must go after positional args.
Either of issue #4188 or issue #15398 would solve this problem for when the optional args are named.
But the following proposal (this issue) solves this problem for when the optionals are positional:
Allow trailing required positional parameters:
task(String name, [List<String> dependencies = const []], callback()) {
    // ...
}

task('init', () {
    // Long
    // code
    // block.
});

task('build', ['init'], () {
    // Long
    // code
    // block.
});

The best we can currently do is:
task(String name, depsOrCallback, [callback()]) {
    var deps = callback == null ? const [] : depsOrCallback;
    if (callback == null) callback = depsOrCallback;
    // ...
}

which:

Loses the type (and thus documentation and checked-mode safety)
Loses the default value (and thus documentation and auto-initialization)
Requires extra logic to assign the parameters correctly.

Duplicate of #15398.


Compatibility with chat completions endpoint

Adjust code to make it compatible with the chat completions endpoint

Hi all. Thank you for the contribution. It is not clear to me how these changes are solving the problem described. Closing for now.


//test/cpp/end2end:xds_end2end_test is way to slow

This is very likely the root cause of #20163 and #20695.
The current runtime of the test under TSAN is around ~2200 sec. That's not a reasonable test duration at all (it's bottle neck for our test latencies both on CI and when running tests manually) and as a side effect we're hitting some test infrastructure corner cases that are masking the underlying problem (e.g. #20695)
The evolution of the test duration (on TSAN, with other configurations there's a similar pattern):

before 2019-10-17 the duration was ~750sec (already pretty high):
https://source.cloud.google.com/results/invocations/d4e77a6c-a1d6-4523-98d8-a5f1d6cfdab0/targets;query=xds_end2end

after merging https://github.com/grpc/grpc/pull/20476 it doubled to ~1500sec
https://source.cloud.google.com/results/invocations/7144d3a9-c005-4154-b20b-a5ab9f9c3ad1/targets;query=xds_end2end_test

then it further increased to ~2200sec (I wasn't able to track down the rootcause).
https://source.cloud.google.com/results/invocations/ef0551a3-143f-4751-bc26-9fe41a1202d7/targets;query=xds_end2end

I was able to confirm that https://github.com/grpc/grpc/pull/20476 has almost doubled the runtime for some of our RBE jobs on kokoro (e.g. UBSAN).

I skimmed through the test logs and saw these logs (notice the big differences in timestamps - sometimes 5mins!)
 Sending response 'resources {

I1023 22:39:09.601898015      18 xds_end2end_test.cc:716]    ========= WAITING FOR BACKEND 1 ==========
I1023 22:44:09.539344910     373 xds_end2end_test.cc:459]    ADS[0x2174698]: Woke up! Sending response 'resources {

I1023 22:45:45.062650763      18 xds_end2end_test.cc:1647]   ========= DONE WITH FIRST BATCH ==========
I1023 22:46:34.366399264     702 xds_end2end_test.cc:459]    ADS[0x219b598]: Woke up! Sending response 'resources {

All are from this log:
https://source.cloud.google.com/results/invocations/38cec024-3389-4810-9552-0937a0c610be/log

Worked around by https://github.com/grpc/grpc/pull/21274


NQueens.roc: segmentation fault when input is not a number

When I run "examples/benchmarks/NQueens.roc" and type "h", it says "Segmentation fault (core dumped)"

@ayazhafiz Perhaps we shouldn't fix this, as today the benchmarks are being reclassified to a less user-facing, more QA-only role.


Cannot handle struct "values"

I am not sure what's the technical name for declaring a structured value like this:
stuff : struct {
blabla: float;
}
But Jails cannot understand them and can't complete nor navigate to the members;
Here's the simple repro code:

stuff : struct {
    stuff_a: float;
    stuff_b: float;
}

StuffStruct :: struct {
    stuff_c: float;
    stuff_d: float;
}

mystuff : StuffStruct;

main :: () {
    print("%\n", mystuff.stuff_c); // It will complete and navigate

    print("%\n", stuff.stuff_a); // It won't complete nor navigate to "stuff_a", it will navigate to "stuff" though

}

Environment: VSCode, Jails 0.0.11, Jai 0.1.096

Sent a PR (#35) with an easy fix to this.

Resolved in #35.


Added fixedDragMeshBoundsSize for boundingbox gizmo to size anchors…

… from bounds size
Follow up https://forum.babylonjs.com/t/gizmo-bounding-box-scale-problem/15354/5

LGTM, maybe you should document that fixedDragMeshScreenSize takes precedence over fixedDragMeshBoundsSize if both are true?


Correct checkbox value change issue #345

Resolve issue with needing local variables in order to update state when a checkbox value is changed
Updated based upon https://chrissainty.com/a-detailed-look-at-data-binding-in-blazor/ "Two Way Binding Between Components" section.

@DouglasRiddle  Thanks for the PR. Please review the current code as at some point two-way binding was implemented by @SamProf.  Please re-open if this has not been resolved.
https://github.com/SamProf/MatBlazor/blob/master/src/MatBlazor/Components/MatCheckbox/BaseMatCheckboxInternal.cs


Plugin to work with multi source argo deployment

Is your feature request related to a problem? Please describe.
ArgoCD now supports creating an application using multiple sources e.g.
apiVersion: argoproj.io/v1alpha1
kind: Application
spec:
  sources:
  - repoURL: 'https://prometheus-community.github.io/helm-charts'
    chart: prometheus
    targetRevision: 15.7.1
    helm:
      valueFiles:
      - $values/charts/prometheus/values.yaml
  - repoURL: 'https://git.example.gom/org/value-files.git'
    targetRevision: dev
    ref: values

Using multi-source together with the plugin doesn't seem to work.
Describe the solution you'd like
The way I'd like this to work is having the AVP work together with the multisource e.g.
apiVersion: argoproj.io/v1alpha1
kind: Application
spec:
  sources:
  - repoURL: 'https://grafana.github.io/helm-charts'
    chart: grafana
    targetRevision: 6.53.0
    plugin:
      name: argocd-vault-plugin-helm
      env:
        - name: helm_args
          value: '-f $values/dev-values.yaml'
  - repoURL: 'https://git.example.gom/org/value-files.git'
    targetRevision: HEAD
    ref: values

If this can work together in some other way That would be great too.

Duplicate of #458


Lower working hours requirement

[x] Determine whether minimum for full time should be 32, 30, 24, 20, 18, or 16 hours
[x] take into account the desirability of couples both working here being able to have no overlapping work hours

[x] consider making weekly all-hands an automated newsletter/podcast for full async

[x] Consider need for work to take up no more than 8 hours daily including 1 non-work lunch hour
[x] consider that such would require 2.5 working days totalling 20 hours subtract 2 hours of lunch breaks, 3 for consistency
[x] consider possible schedules for 16 and 18 hour weeks, weigh convenience against productivity
[ ] determine whether 2 or 3 days weekly minimum, balancing flexibility with gantt delay
[ ] determine whether higher minimum across teams and projects balancing agency and gantt delay
[ ] change requirement to 16 hours weekly
[ ] change all-hands to async
[ ] stress that this number is a minimum, that no one is to feel pressured or obligated to do more, nor chastened for doing more.
[ ] adjust share count to 100 times weekly minimum hours (1600)

Current proposal combined with hours suggested in #15 would result in 100 employee cap being equivalent in annual working hours to 35-person traditional company. Need plan to ensure smooth functioning, growth, and revenue inflow unimpeded by working hours plans before drafting and merging either issue.
Determine plan and draft articles outlining how to keep company operating smoothly with sparse asynchronous hours
Draft Hours policy accommodating of both operational plan and employee flexibility goals
Draft Vacation policy accommodating of both operational plan and employee flexibility goals
Merge all three articles in same PR.


Error Data.Length in IntroSlider.JS

Hi,
After updating the Expo version, the following error is returning in the component. Can you help me please?
**TypeError: undefined is not and object (evaluating: 'data.length')
This error is located at:
in AppIntroSlider (at IntroSlider.js: 135)**

Hey man. Change to new version slides in AppIntroSlide to data like this

Hi, it worked .. Thank you so much !!
But buttonStyle is no longer working, did you change the name of the props?

But buttonStyle is no longer working, did you change the name of the props?

buttonStyle has been removed in favor of using the renderDoneButton/renderNextButton etc. functions. You can find out more about the release under release notes.
Hope it makes sense!

Closing this since it seems to be resolved


Alphabetical ordering in facets affected by casing

Bug Report
Expected behaviour
In this case, I would expect casing to be ignored.
Actual behaviour
Case affects the order in the alphabetical list.
Steps to reproduce the behaviour

Go to https://staging.dockstore.org/
Click alphabetical to sort by in the Author facet
Notice n/a appears before Aaron.
The same problem happens in the search results when clicking on the header to sort

┆Issue is synchronized with this Jira Story
┆Issue Type: Story
┆Fix Versions: Dockstore 1.8
┆Sprint: Seabright Sprint 24 Yacht
┆Issue Number: DOCK-1030
┆Epic: Fix Facets

Working in UI - 2.5.0-alpha.0 on staging.


(Chore) #822 splash screen remove texts

Description
Remove top text on splash screen.
About #822
How Has This Been Tested?
Just open the app and while it is loading you will see the splash screen.
Checklist:

[x] PR title matches follow: (Feature|Bug|Chore) Task Name
[x] My code follows the style guidelines of this project
[ ] I have followed all the instructions described in the initial task (check Definitions of Done)
[x] I have performed a self-review of my own code
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I have added reference to a related issue in the repository
[x] I have added a detailed description of the changes proposed in the pull request. I am as descriptive as possible, assisting reviewers as much as possible.
[x] I have added screenshots related to my pull request ( for frontend tasks)
[ ] I have pasted a gif showing the feature.
[x] @mentions of the person or team responsible for reviewing proposed changes

Screenshot:

Waining for @LiavGut's response to my question in related issue comments.


Application is crashing when started with -unattended (LV2021)

After updating to LabVIEW 2021 32-Bit I ran into a problem I've no encountered jet.
I have an single VI build into an application that is using the G-CLI. I'm using this command to run my application:
g-cli --kill --verbose "{path to my .exe"} -- -b "{a path my application needs}" so nothing special here.
When using LV2020 everything is fine but when using LV2021 the application crashes immediately with the following error:

The log file is almost empty and useless to my knowledge. Even when disabling the entire content of the VI with a diagram disable structure it crashes the same way.
After trying a few things I came up with a temporary solution. When the G-CLI is starting my application it does it with the -unattended flag which normally is a good thing cause we don't want any dialogue windows when running an application in a CI pipeline. But this flag seems to be the cause of the crash. In my application I set this flag to true anyway so this is not a problem.
Is there a way to disable the flag for now so I can still run my pipeline until there is some kind of fix for this?

Hi @McGeifer
Thanks for reporting this. I was wondering whether we need an option to remove that flag and you seem to have confirmed it!
If you add the flag --allowDialogs to the G CLI portion this should remove the unattended flag. Let me know if that fixes the issue and we can try and put something together as a bug report to NI.

Hi!
--allowDialogs is working perfectly fine 👍


Runelite Display Scaling Feature Request

Please find an example below whilst showing when using the option to change my display scaling on Windows 10 from 100% to 175% whilst using Konduit & Runelite. I have both Konduit and Runelite open to show how this feature appears on both clients. I know this feature request is supported on the official client & Konduit, however, this would be great add-on for Runelite too.

This changes the chatbox, inventory, and minimap to appear larger when playing in resizable mode. I feel this would be a great feature to allow easier playgame and less drag time when interacting with the mouse. I personally enjoy playing on resizeable and find it's quite the laid back casual gaming approach. However, I have bad eyesight and often find it difficult to read the chat box, especially when speaking with friends over PM. I would recommend trying this feature request out for yourselves on the official client too.
I would appreciate and encourage any form of feedback against this request.

This was once upon a time being worked on as an official feature by Jagex:
https://twitter.com/MrIanGower/status/607698751482765313

I'd personally love to see this as well. (ignoring the fact that its OSHD aspect) , https://twitter.com/OSHDcom/status/873264929138147328 , They seemed to have a nice way to set different levels of scaling via the client.

This is actually quite a bit more difficult than it sounds, especially now that the Deobbed client is closed source. Maybe some day

I believe we added support for scaling already. Are you using the installer?

Duplicate of #858


Add Ignition support

We want to add support for Ignition as an additional bootstrap mechanism for CoreOS setups.

[x] Add a use-ignition argument to the cli
[x] Test the template generation
[x]  Test on a real AWS cluster

Ideally, we could try to gather the CoreOS version of the AMI to verify if it supports ignition.

only minor comments ;) nothing major

@zeisss Do you need someone else review to give me a LGTM ? I mostly addressed all your comments.

ping @zeisss for an answer

LGTM, if it works for you, feel free to merge.

@zeisss It works but I found a weird behavior with the etcd cluster creation. I need to test it more

@zeisss It works and the issue of the etcd out of the box creation. It also happens with cloudconfig setup so it should be a different thing and totally independent of this PR.

still lgtm ;)