Use default import path in crossbuild or abort

The detected repo path in promu info is correct. It's not used in crossbuild though.
docker run --rm -t -v $PWD:/app prom/golang-builder:1.5.3-main -i  -p 'linux/amd64 ...
Instead of passing in an empty variable, promu should at least abort with something like unknown repo URL, but even better use the default provided by repoLocation().

The thing about setting repoLocation() as default value for repository.path is that it won't work with forks.

One way to find the correct import path would be to use Go 1.4 annotation to identify the canonical import path.
Therefore, by having this  comment in your code:
package main // import "github.com/prometheus/node_exporter"

It will let us do: go list -e -f '{{.ImportComment}}'  to recover the correct project import path.

That looks like a great idea.
What is the use case of the current repoLocation() function then? It seems prom build uses it currently, but this will equally fail if someone forks a project and tries to run promu build, no?

promu build doesn't use repoLocation() directly. It uses the ProjectInfo struct which is partially initiated by the repoLocation() func (https://github.com/prometheus/promu/blob/master/cmd/info.go#L50). So repoLocation() is the source of information for Owner, Name and Repo properties.
Currently promu build only uses Version and Name of that struct.
However, Owner and Name are used by the promu release command (https://github.com/prometheus/promu/blob/master/cmd/release.go#L66), which is correct when using a fork.

I just realized, the canonical import path annotation will not work great for prometheus/prometheus because of multiple binaries.

Alright, then let's make it a requirement to set in .promu.yml.


Fix VanillaItems::AIR()

It's not necessary to use VanillaBlocks->asItem(). We already have VanillaItems in PMMP


Provide support for arbitrary file extensions within Dash for R component libraries

This PR proposes to address #185, which implements functionality found in plotly/dash#481. In particular, the internal get_mimetype function should now handle both .map and .js.map, and uses guess_mimetype from the mime package for extensions other than .js, .css and .map/.js.map.
This feature will require edits to Dash's package generator; a PR proposing those changes is in process.
If the extension is unknown, Dash for R will assume application/octet-stream instead. Other features this PR would provide:

support for loading binary file dependencies within the _dash-component-suites handler
support for properly setting a variety of new resource$type values via the mime package

This PR is currently on hold until minor edits are made within Dash for component library assets, as noted in https://github.com/plotly/dash/pull/1186.


#163002854 User can view meetups

What does this PR do?

Creates a meetups page meetups.html

Description of the task to be completed

[x] Creating meetups page

How can this be manually tested?

Go to the repo and clone it
Navigate to the folders
Using your favorite browser, run file index.html
Login to view meetups

What are the relevant PT stories
#163002854

looks good...probably add a border to physically seperate each meetup from the other..

Thanks @wachiranduati  I will try and add that border

Looks good and yeah a border would do some good.


fix(Modal): add support for primary focus selector prop

Closes IBM/carbon-components-react#1123
This PR will allow users to provide a selector that will be focused when the modal is opened. This behavior will be in line with the vanilla component's capabilities.
Changelog
New

add selectorPrimaryFocus prop on <Modal>
include <ModalWrapper> Storybook example showing selectorPrimaryFocus prop behavior

Changed

modify auto focus behavior based on selectorPrimaryFocus value, or use default primary actionable item

I suggest, that the data attribute should also be possible to set in other carbon components like ComboBox. They contain text fields, which should also be focusable in modal views.


Remove deprecated device context fields

There are a couple of fields which are deprecated in the device context. Those fields should be removed in v7.0. Follow up from #838

Here's the list:
device.language
device.timezone
device.screenResolution
device.theme

Speaking of removing deprecated fields, there are a couple more candidates:

https://github.com/getsentry/sentry-dart/blob/6ae0ef03535461e6a75d1eb297e71376eaeed749/dart/lib/src/enricher/io_enricher_event_processor.dart#L95-L96 since it was moved to event.thread.name and can be enabled by options.attachThreads
This runtime key https://github.com/getsentry/sentry-dart/blob/6ae0ef03535461e6a75d1eb297e71376eaeed749/dart/lib/src/enricher/io_enricher_event_processor.dart#L58 since it doesn't add much value as name is already indicating what it is
Same for this key https://github.com/getsentry/sentry-dart/blob/6ae0ef03535461e6a75d1eb297e71376eaeed749/flutter/lib/src/flutter_enricher_event_processor.dart#L224 since it doesn't add much value as name is already indicating what it is

https://github.com/getsentry/sentry-dart/pull/934 introduced attaching http response data, so the MaxRequestBodySize and MaxResponseBodySize enums can be consolidated.


implement relation

Fixes https://github.com/ongr-io/ElasticsearchDSL/issues/220

I'll make a new PR which includes the tests


feat: add mockClipboard action

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[x] Feature
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other (please describe):

Issue Number: WEB-6124 Перенести e2ed/utils/mockClipboard.ts в экшены e2ed

Here we need to change the implementation in the client function (in this form it will not work, it seems), and add tests for this action.
So we will add this functionality later.


Send the repartition summary to someone

As we do not store anything, it could be nice to propose to send a summary to someone.

Yes, only "on demand", with an option.
When running the app, the admin could ask for the list.
Maybe it could be send in a SlackBot reminder? To be really visible only after a certain date?

Maybe a protected PDF?


Latest release is currently broken / jsxlate-loader no longer functioning

I'm trying to integrate this component into my project, along with jsxlate-loader, and I've run into a few issues:
index.js has several invalid imports:
var translator = require('./lib/jsxlate');
var components = require('./lib/components.jsx');
var cache = require('./lib/cache.js');

The lib directory isn't present because the build.sh script isn't currently working. It looks like it creates the directory, but then puts the files into ./build instead. Even with changing the path, the first file doesn't exist anymore.
Looking at jsxlate-loader, it seems like it still relies on the 0.3.x release. Should I just not use the latest release that's been published?

I spent a few hours working on this and I was able to get things working with the older versions, though it took some finagling because Webpack wasn't too happy about the way imports were structured in the examples (they were including node-only modules).
I'm really excited to use your library at my company, and I'd love to get a sense of what the future of the project is, or whether it would make more sense for me to fork the older version. I'm using it to provide translations for an isomorphic React app and there are currently quite a few hurdles to getting it integrated. I'd love to help in any way I can.

Sorry for the ham-fisted release. I didn't think anyone was actively using it or I would have been a bit more careful... 😅
So, jsxlate 0.4+ is now based on Babel 6 and does not require jsxlate-loader. To use jsxlate you simply list it in your .babelrc plugins list. I intend to have some examples up soon. If you're still using Babel 5 I would recommend pinning to the old version.
If you let me know what integration difficulties you're having I'd be happy to help address them as best I can. We have been using jsxlate in production for several months without issue, in an isomorphic app as well, at idealist.
Sorry again for costing you so much time — I'm thrilled you're interested in the project!

Oh, meant to mention: index.js obviously needs an update. I should have a new version out tomorrow.

@drd Thanks for the quick reply! It's actually been a fun challenge getting up and running, so no worries at all. I actually just realized that I could add it to .babelrc once I read through the src README, did that, and it's working like a charm.
Here's another issue I ran into: babel-polyfill isn't included in any of the build files, and as a result my Webpack compilation will only work if I require it in the config file. What are the odds that you might be able to automatically include it since it's already a package dependency? I'm not sure if there's another better approach to this, but because it's not present, Webpack fails on a few calls to Object.entries and trying to call toJSON() on a set.

Just a little more context... even when running Babel from the command line, I get the same error:
$(npm bin)/babel app/shared/apps/repair/components/ColorsContainer/ColorsContainer.js
TypeError: app/shared/apps/repair/components/ColorsContainer/ColorsContainer.js: variables.toJSON is not a function
    at freeVariablesInMessage (/Users/steven/dev/icr-front-end/node_modules/jsxlate/build/free-variables.js:23:22)
    at Object.transformElementMarker (/Users/steven/dev/icr-front-end/node_modules/jsxlate/build/transformation.js:38:48)
    at PluginPass.enter (/Users/steven/dev/icr-front-end/node_modules/jsxlate/build/transform.js:19:67)
    at newFn (/Users/steven/dev/icr-front-end/node_modules/babel-core/node_modules/babel-traverse/lib/visitors.js:262:19)
    at NodePath._call (/Users/steven/dev/icr-front-end/node_modules/babel-core/node_modules/babel-traverse/lib/path/context.js:63:18)
    at NodePath.call (/Users/steven/dev/icr-front-end/node_modules/babel-core/node_modules/babel-traverse/lib/path/context.js:47:17)
    at NodePath.visit (/Users/steven/dev/icr-front-end/node_modules/babel-core/node_modules/babel-traverse/lib/path/context.js:93:12)
    at TraversalContext.visitQueue (/Users/steven/dev/icr-front-end/node_modules/babel-core/node_modules/babel-traverse/lib/context.js:146:16)
    at TraversalContext.visitMultiple (/Users/steven/dev/icr-front-end/node_modules/babel-core/node_modules/babel-traverse/lib/context.js:110:17)
    at TraversalContext.visit (/Users/steven/dev/icr-front-end/node_modules/babel-core/node_modules/babel-traverse/lib/context.js:176:19)

For reference, here's my .babelrc:
{
 "presets": [ "react", "es2015" ],
 "plugins": [ "jsxlate" ]
}

I did a little searching, and it looks there isn't a preset that includes Object.entries or map/set.toJSONyet.

Yes, that is something that must be included using babel-polyfill, which should not be require() by a library — only one instance of it can be active at a time, so it is your app's responsibility to include it. It should be marked as a peerDependency and documented. Thanks for pointing this out!

Thanks, Eric! I'm happy to add babel-polyfill as a peerDependency in a PR and update the documentation as well, if you'd like. Just let me know. Feel free to close this whenever you'd like.

Yes, that would be awesome. I just pushed some updates that attempt to clarify the situation, but I didn't add the peerDep, if you could that would be awesome!

I'm happy to do this, but how would you explain in the README what people would need to do to run babel-cli using the jsxlate plugin? Does babel automatically import it? Otherwise, I'm not sure how you would get it to work.

I think the only thing you need to do is have require('babel-polyfill') in the entrypoint of your app. This will probably be true for tests as well. We have it in server, client, and test entrypoints. I'm not sure of a better way to handle this.

Got it, but my concern is that your local jsxlate tests won't pass if babel-polyfill isn't a hard dependency. I'll give it a shot and see what happens.

Ha, apparently it already is: https://github.com/drd/jsxlate/commit/9a258c5cbe041d62917a487814bd0983d9ed7bdd
And it's required in test/test-extraction.js...

That's why I was asking if you really want to move it to peerDependencies. I'm guessing you really just want to add it, not move it, which I'm happy to do, but I'm not sure what it buys, since any project including the library is guaranteed to have babel-polyfill as a result of you including it, no?

 Honestly I'm not even sure what the best thing to do is until everyone is on npm v3. I guess let's leave it as a hard dependency and just mention it in the docs that it's required to be added to the entrypoints of your app.

The problem is that if I include babel-polyfill in the library code, and someone else's app is already including babel-polyfill, then it will error. See https://github.com/babel/babel/blob/master/packages/babel-polyfill/src/index.js#L3

That's odd -- I was including it but didn't run into this problem. But maybe that's because I'm using npm 3 and the versions match? This makes my head hurt. Haha

I have run into the same issue. According to this: https://github.com/DavidBruant/Map-Set.prototype.toJSON/issues/16 toJSON for Set and Map is rejected so it would be better to use something else for serialization.

@aleksey-shmatov thanks for the heads up about toJSON. I have switched to using an Object as the set. @stevenp I have completely removed jsxlate's dependency on babel-polyfill so hopefully that will help! Sorry this took so long, I just started a new job … and we don't use jsxlate (yet!). Let me know if 0.6.0 resolves your issues.


Weaving Angular config and classes

What is your guidance about weaving Angular configuration such as registering controllers, services, etc. with the Angular module system and defining classes?
As importing classes and typings should not have side-effects, cf. https://github.com/Microsoft/TypeScript/wiki/FAQ#why-are-imports-being-elided-in-my-emit
I am trying to set up a practice of bundling the loading of Angular configuration in a couple of hierarchical ordered TypeScript files that are imported by the syntax:
import "modulename";
which will enforce the execution of the static code within it. In some cases this requires me to setup double .ts-files:

mycode.ts; imported by import {myclass1, mytype1} from "./mycode";
mycode.angular.ts; imported by import "mycode.angular.ts";

This does not look like the best solution. What is your guidance?

I"m not sure what you mean with your question.
If you mean register controllers and services with angular, I always do it in an index.ts file for every angular module, like so
`import {SharedModel} from "../core/models/impl/SharedModel";
import {SidebarModel} from "../core/models/impl/SidebarModel";
import {TopbarModel} from "../core/models/impl/TopbarModel";
import {TweetService} from "../core/services/impl/TweetService";
angular.module("app.tweets", ["ngRoute"])
.service("ContentModel", ContentModel)
.service("SharedModel", SharedModel)
.service("SidebarModel", SidebarModel)`
If you want to export angular modules you can do it like so:
export var tweetMod: IModule = angular.module('app.tweets')....
and use it like this:
import {tweetMod} from './tweets/index'; angular.module('app', [tweetMod.name]);
Does that answer your question?

Since no further comments, I'm going to close this issue.


Add proxying component to profiling subsystem

This PR is the first of two PRs implementing #549. It introduces a new HTTP handler to our profiling subsystem that can be used to collect and join sensors from a dynamically resolved list of endpoints.
There are no unit-tests in this PR. An integration test validating this functionality is present in the second PR.

Shipped!

Shipped!


General feedback about Strapi 5 support

I'm creating this general issue to collect some feedback from other people about the work done in past PRs to support Strapi 5 and how we can improve it. If you want it, maybe I create separate issues to track these individually.
After installing the module myself and using v5 for a while in a personal project I've found the following:

Maybe It'll be good to document that if people want to try v5 out, aside from switching to the edge version, they should also set the version manually in the nuxt.config.ts file:export default defineNuxtConfig({
  strapi: {
    version: 'v5',
  },
})

For some reason, if you pass an interface instead of a type as the generic type of the useStrapi<T>() composable it will not give you the correct types. This didn't happen to me on the v4 composable and I don't even know if this is fixable (because it seems to be a TS thing) but I guess at least we should tell people to use types instead of interfaces in the docs, or even better, try to find a workaround. In my opinion, users should not be bothered with this and they should be able to use both if they want. I've created a small repro here: https://tsplay.dev/WYxeQm.
If I do a find request to a collection to query all elements, the locale key has a null value, which differs from the current type locale?: string, I think it should be locale?: string | null instead:

For now, that's what I've found. Let me know what you think!
Related PRs:

https://github.com/nuxt-modules/strapi/pull/424
https://github.com/nuxt-modules/strapi/pull/425
https://github.com/nuxt-modules/strapi/pull/426

@olrtg isn't it supposed to be:
export default defineNuxtConfig({
  // Example of separate client/server URLs
  runtimeConfig: {
    strapi: {
      version: 'v5',
    }
  }
})

maybe
same
agree

Yes, the fact that v5 is not the default should be somewhere in the docs. Could be tricky as the default version of Strapi is now 5. The module has not been fully tested for v5.
I am no TS expert so I am not even sure how to approach this 🤣. That is a weird one tho. Why would types work and not interfaces?
Correct! locale can be null.

@reslear would the StrapiLocale be a composable? Not sure what you mean there.

As you guys might know, I'm not actively maintaining this repository. Would you mind telling me once everything is good so I can release a major version?

✅ 1, 3 - fixed
About No2 i was thinking maybe we could just do it like v4 without generics ?
like data | data []

@reslear 2 sounds great to me. About the other points idk. I just started using this module and Strapi, so I don't know much yet.

Hi guys, I'm trying to use useStrapi module (V5), nuxt can't find useStrapi, however I can use useStrapiUser, UseStrapiToken, useStrapiAuth, etc... do you know why?

@Zellokr Do you use the latest Edge version?

Yes, as devDepencencies

Could you try npm<EMAIL_ADDRESS>(latest release), sometimes the latest tag don't work that well.

it works, however type error "V5" in version is still appears.
Thank you! :)

✅ 1, 3 - fixed

i was thinking maybe we could just do it like v4 without generics ?
like data | data []

I'm also thinking of exporting types like
import type { StrapiLocale } from '#nuxt-strapi/types';

I also keep the same types between projects, I suggest adding StrapiDynamicZone, StrapiComponent, StrapiImage, StrapiSeo.

For dynamic editor I suggest to add information about

https://github.com/niklasfjeldberg/vue-strapi-blocks-renderer
https://github.com/freb97/nuxt-strapi-blocks-renderer

#3 sounds great. With the latest i can't import any types from "@nuxtjs/strapi", i.e. import type { Strapi4ResponseData } from "@nuxtjs/strapi"; This did work in strapi v4, but seems broken now.

Stopping by here from the Strapi team (as I work for Strapi) to mention that if anyone needs any information clarity on how something works or help maintain or build support for Strapi 5 please don't hesitate to stop by our Discord: https://discord.strapi.io
Or if you are in dire need of some information for this module you can email me directly<EMAIL_ADDRESS>(please don't send emails asking for normal application support but if it's related to this module then it's all good).
Thanks!

Just released a v2.0.0, let me know if you encounter any issue with it!

Could you guys have a look at https://github.com/nuxt-modules/strapi/pull/445? Thanks!

Find a small deprecated part in the documentation, might as well post it here so it can be considered to change:
https://strapi.nuxtjs.org/advanced#file-upload is the old way which is not supported by strapi v5 anymore as stated per: https://docs.strapi.io/dev-docs/migration/v4-to-v5/breaking-changes/no-upload-at-entry-creation
I am currently working on updating my applications which use this piece of code so if I find some time in the near feature I could chip in to change the documentation having a section for the old v4 way as a reference and the new way.


feat: supported multicall on v1 mainnet

supported multicall to avoid https://github.com/nervosnetwork/light-godwoken/issues/136 ,this multicall contract may replace with https://github.com/mds1/multicall/issues/33 in the future

This pr reduces rpc call frequency significantly when fetching erc20 token balances.

@homura @zhangyouxin multicall3 deployed on mainnet https://github.com/mds1/multicall/issues/33


A problem in the examples

Is this a BUG REPORT or FEATURE REQUEST?:
I saw the example that you given in examples\python, but a failure i missed. In the file called hellowithdeps.py, you have used the code ' from hellowithdepshelper import foo'. Do you mean that i can call another function in one function?

Hi @Huangxiangjie, you can import local files from your function as far as you zip them in a single file. You can also include external dependencies (pip packages) if you include a requirements.txt file as explained in the quick-start guide: https://kubeless.io/docs/quick-start/

Hi @Huangxiangjie, you can import local files from your function as far as you zip them in a single file. You can also include external dependencies (pip packages) if you include a requirements.txt file as explained in the quick-start guide: https://kubeless.io/docs/quick-start/

Do you mean that two function in one file？

Hi @Huangxiangjie, you can import local files from your function as far as you zip them in a single file. You can also include external dependencies (pip packages) if you include a requirements.txt file as explained in the quick-start guide: https://kubeless.io/docs/quick-start/

Thank you. But when i creat functions through zip file, the function is messy code.


GRG Widget cell size in creation

Widget
GRG Widget
Version of widget
1.0
Bug or Enhancement
Bug
Repo Steps or Enhancement details
We have attempted to create a graphic that covers an area that should have each grid be 100 X 100; however, this is what we get.

Same size issue with hexagonal grid:

@topowright issue confirmed. Will get with gigz on this.

Here is an example when creating a grid further away from the equator:

Talking to some other people in Solutions (@MikeMillerGIS and @pLeBlanc93  )there is a thought that it could be a difference in using planar measurements and not geodesic measurements.

Here is the measurement of a 1000m grid using the geodesic measurement in ArcGIS Pro:

@topowright We had a very similar problem when applying planar buffers (with Public Notification) and ground measurements (using Parcel Drafter) on web-mercator basemaps.  Could it be that the measure tool is actually reporting planar measures?

@topowright Confirmed: The measurement widget (which relies on esri/dijit/Measurement) by default measures in geodesic mode. The grg widget will need to generate geodesic grids.

@ChrisBuscaglia  and I used ParcelDrafter to make a 100m x 100m square, then added it to the app's basemap. Unfortunately the GRG built a less-than-100m square grid. So the Measure widget is correct, GRG is not.

This issue was moved to ArcGIS/solutions-grg-widget#5


CPU usage spikes with JS files, especially when opening several quickly from Search results

Issue Type: Bug

Launch VS Code with a NodeJS project open (it's a create-react-app project).
Wait for initial CPU usage to come down after launch.
Perform a Search that results in about 10 JS files.
Double-click each JS file to open it, then close it, then proceed to the next one.
CPU usage spikes (see screenshots). Most of this is due to Code Helper processes, and can be seen in VS Code's Process Explorer, under the "electron_node tsserver.js" process.
Wait for CPU usage to come down; this takes over five minutes.
Repeat the same Search, and open each file again. CPU usage spikes again for over five minutes.
During the spike, the message "Initializing JS/TS language features" appears in the status bar. Sometimes it disappears then reappears.

I submitted log file ID<PHONE_NUMBER> privately. It was recorded while doing the steps above. The attached process-explorer.png image shows the Process Explorer during one of the spikes. The attached activity-monitor.png shows the CPU usage during the spikes (you can see the two points where CPU usage came down).
(I know you'd prefer a report with extensions disabled. I'll try to get one, but it's been hard to get a good reproduction, so I thought I'd submit this first.)
This is using the built-in TypeScript 3.2.2.

VS Code version: Code 1.30.0 (c6e592b2b5770e40a98cb9c2715a8ef89aec3d74, 2018-12-11T22:22:15.793Z)
OS version: Darwin x64 18.2.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-3615QM CPU @ 2.30GHz (8 x 2300)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: enabledrasterization: unavailable_softwarevideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
2, 4, 5

Memory (System)
16.00GB (1.69GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (21)

Extension
Author (truncated)
Version

project-manager
ale
10.0.0

vscode-intelephense-client
bme
0.8.8

npm-intellisense
chr
1.3.0

dart-code
Dar
2.21.1

mustache
daw
1.1.1

vscode-eslint
dba
1.7.2

vscode-npm-script
eg2
0.3.5

php-debug
fel
1.12.6

svn-scm
joh
1.46.4

vs-script-commands
mkl
7.0.0

vscode-apache
mrm
1.1.1

python
ms-
2018.12.1

csharp
ms-
1.17.1

debugger-for-chrome
msj
4.11.1

java
red
0.35.0

vscode-java-debug
vsc
0.16.0

vscode-java-dependency
vsc
0.2.0

vscode-java-pack
vsc
0.5.0

vscode-java-test
vsc
0.12.0

vscode-maven
vsc
0.12.1

jinja
who
0.0.8

Also having problems with the extension host hitting CPU hard although I'm not sure if it's related. Started happening two updates ago. No specific extension according to the process explorer:

Restarting VS code isn't helping. It stops after certain time. Can't even type source code when it's happening.

@mareksuscak open a new issue

@jacobweber Can you please test this again on the latest vscode insider build. I could not track down what the root cause was but it's possible that it may since have been fixed

Yes and no. There are so many things that seem to cause the CPU to spike on a large JS/React project. I can't narrow it down. But I have seen what seems like the same issue on 1.33.1.

@jacobweber Can please you upload a new copy of the logs using the latest VS Code insiders build

Without more information, I'm closing this as possible the same issue that we see in https://github.com/microsoft/TypeScript/issues/31452
Please test this again with the latest typescript@next by following these instructions. If still see this issue and have good repo steps or can share a small example project, let me know


docs(dynamic-cookbook): upgrade to use new forms api

I have added a version of the Dynamic Form Cookbook using the new forms API.

I like it.

This is a bit of a nitpick, but could you please put your imports in alphabetic order and leave an space between angular imports and your own imports? You are doing that for almost all files but a few :)
Thanks!

Yep. I just pushed better ordering of imports


Refactoring of the Logger

Short Description:
I would like to use python's logging lib over the custom functions which are sometimes locally defined, having different names, etc.
Changes:

Added ColorLog Dependency
Removed all local _log or log functions
Logger now using Debug, Info, Warning, Error and Critical for messages and coloring
Custom colors for the message's can still be provided
Refactored all files to use the new logging methods
Removed LogByEvent Test and FireEvent on Log-Message (is this ok?)
Using TimedRotatingFileHandler FileHandler for log file rotation
Added option for logging to file in the config
Customizable Date in the Filename and it's possible to seperate the files into different directories on a per user basis
Anjusted the Tests

Tests:
Please have a look at the tests, because im not sure if it's alright like this :)
Preview:

@OpenPoGo/maintainers
I still need to have a look though all messages and set them on the right level, most of them are using info at the moment.

Current coverage is 51.18% (diff: 54.32%)

Merging #344 into master will decrease coverage by 0.07%

@@             master       #344   diff @@
==========================================
  Files            34         34
  Lines          1500       1555    +55
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            769        796    +27
- Misses          731        759    +28
  Partials          0          0

Powered by Codecov. Last update acbc124...6268ab4

This is still unready.

It's ready now.
I added a deprecation notice, to inform the users about the added logging-sector in the config.yml - It still work's without adding it though.
If you merge this, #346 will also need to be changed to the new logging methods.

This is nice. I can use file logging on my stealth branch as I need some more testing and reporting.

It's ready.
@OpenPoGo/maintainers

:+1:


UI changes needed

[ ] Change the background from powderblue to maroon?

[ ] Could we increase the margin between the tic-tac-toe board and the scoreboard? A 5% increase or something should suffice!

ack


Add current recipe name to the title

Fixes #200.

Great :+1:
I'd suggest flipping the two parts, though. The title is normablly abbreviated in the tab strip and having the content first and then the site name helps in skimming through the tabs.

Done!


Fastlane hangs then crashes with undefined method `prerelease=' for nil:NilClass

I'm trying to learn fast lane, but so far it's not working. My current Fastfile is:
fastlane_require xcodebuild

desc "Increment framework version"
lane :build_framework do |lane|

xcodebuild(
	clean:true
)

end

After updating my ruby to 2.4 and all my gems, fast lane logs this:
[15:42:17]: ------------------------
[15:42:17]: --- Step: xcodebuild ---
[15:42:17]: ------------------------
[15:42:17]: For a more detailed xcodebuild log open /Users/derekclarkson/Library/Logs/fastlane/xcbuild/2017-07-25/92596/xcodebuild.log
[15:42:17]: $ set -o pipefail && xcodebuild  | tee '/Users/derekclarkson/Library/Logs/fastlane/xcbuild/2017-07-25/92596/xcodebuild.log' | xcpretty --color --simple
[15:42:18]: ▸ Building DigitalID/DigitalID-iOS [(Release)]
[15:42:18]: ▸ Check Dependencies
[15:42:18]: ▸ Running script 'Turn tags into warnings'
[15:42:18]: ▸ Build Succeeded
[15:42:18]: Installing Ruby gem '▸ Building DigitalID/DigitalID-iOS [(Release)]
▸ Check Dependencies
▸ Running script 'Turn tags into warnings'
▸ Build Succeeded
'...

Then it hangs for two minutes before crashing out:
[15:44:39]: Found gem "" instead of the required name "▸ Building DigitalID/DigitalID-iOS [(Release)]
▸ Check Dependencies
▸ Running script 'Turn tags into warnings'
▸ Build Succeeded
"

Looking for related GitHub issues on fastlane/fastlane...

➡  live_version not found
    https://github.com/fastlane/fastlane/issues/3489 [closed] 6 💬
    22 Sep 2016

🔗  You can ⌘ + double-click on links to open them directly in your browser.
/Users/derekclarkson/.rvm/rubies/ruby-2.4.0/lib/ruby/site_ruby/2.4.0/rubygems/dependency_installer.rb:470:in `resolve_dependencies': [!] undefined method `prerelease=' for nil:NilClass (NoMethodError)
	from /Users/derekclarkson/.rvm/rubies/ruby-2.4.0/lib/ruby/site_ruby/2.4.0/rubygems/commands/install_command.rb:194:in `install_gem'
	from /Users/derekclarkson/.rvm/gems/ruby-2.4.0@global/gems/fastlane-2.48.0/fastlane/lib/fastlane/fastlane_require.rb:36:in `install_gem_if_needed'
	from /Users/derekclarkson/.rvm/gems/ruby-2.4.0@global/gems/fastlane-2.48.0/fastlane/lib/fastlane/fast_file.rb:190:in `fastlane_require'
	from Fastfile:2:in `parsing_binding'
	from /Users/derekclarkson/.rvm/gems/ruby-2.4.0@global/gems/fastlane-2.48.0/fastlane/lib/fastlane/fast_file.rb:54:in `eval'
	from /Users/derekclarkson/.rvm/gems/ruby-2.4.0@global/gems/fastlane-2.48.0/fastlane/lib/fastlane/fast_file.rb:54:in `block in parse'
	from /Users/derekclarkson/.rvm/gems/ruby-2.4.0@global/gems/fastlane-2.48.0/fastlane/lib/fastlane/fast_file.rb:43:in `chdir'
	from /Users/derekclarkson/.rvm/gems/ruby-2.4.0@global/gems/fastlane-2.48.0/fastlane/lib/fastlane/fast_file.rb:43:in `parse'
	from /Users/derekclarkson/.rvm/gems/ruby-2.4.0@global/gems/fastlane-2.48.0/fastlane/lib/fastlane/fast_file.rb:33:in `initialize'
	from /Users/derekclarkson/.rvm/gems/ruby-2.4.0@global/gems/fastlane-2.48.0/fastlane/lib/fastlane/lane_manager.rb:12:in `new'
	from /Users/derekclarkson/.rvm/gems/ruby-2.4.0@global/gems/fastlane-2.48.0/fastlane/lib/fastlane/lane_manager.rb:12:in `cruise_lane'
	from /Users/derekclarkson/.rvm/gems/ruby-2.4.0@global/gems/fastlane-2.48.0/fastlane/lib/fastlane/command_line_handler.rb:30:in `handle'
	from /Users/derekclarkson/.rvm/gems/ruby-2.4.0@global/gems/fastlane-2.48.0/fastlane/lib/fastlane/commands_generator.rb:104:in `block (2 levels) in run'
	from /Users/derekclarkson/.rvm/gems/ruby-2.4.0@global/gems/commander-fastlane-4.4.5/lib/commander/command.rb:178:in `call'
	from /Users/derekclarkson/.rvm/gems/ruby-2.4.0@global/gems/commander-fastlane-4.4.5/lib/commander/command.rb:153:in `run'
	from /Users/derekclarkson/.rvm/gems/ruby-2.4.0@global/gems/commander-fastlane-4.4.5/lib/commander/runner.rb:476:in `run_active_command'
	from /Users/derekclarkson/.rvm/gems/ruby-2.4.0@global/gems/fastlane-2.48.0/fastlane_core/lib/fastlane_core/ui/fastlane_runner.rb:64:in `run!'
	from /Users/derekclarkson/.rvm/gems/ruby-2.4.0@global/gems/commander-fastlane-4.4.5/lib/commander/delegates.rb:15:in `run!'
	from /Users/derekclarkson/.rvm/gems/ruby-2.4.0@global/gems/fastlane-2.48.0/fastlane/lib/fastlane/commands_generator.rb:303:in `run'
	from /Users/derekclarkson/.rvm/gems/ruby-2.4.0@global/gems/fastlane-2.48.0/fastlane/lib/fastlane/commands_generator.rb:42:in `start'
	from /Users/derekclarkson/.rvm/gems/ruby-2.4.0@global/gems/fastlane-2.48.0/fastlane/lib/fastlane/cli_tools_distributor.rb:66:in `take_off'
	from /Users/derekclarkson/.rvm/gems/ruby-2.4.0@global/gems/fastlane-2.48.0/bin/fastlane:20:in `<top (required)>'
	from /Users/derekclarkson/.rvm/rubies/ruby-2.4.0/bin/fastlane:23:in `load'
	from /Users/derekclarkson/.rvm/rubies/ruby-2.4.0/bin/fastlane:23:in `'
	from /Users/derekclarkson/.rvm/gems/ruby-2.4.0@global/bin/ruby_executable_hooks:15:in `eval'
	from /Users/derekclarkson/.rvm/gems/ruby-2.4.0@global/bin/ruby_executable_hooks:15:in `'

I have no idea what's wrong. What do I fix?

Okies. Here it is:
[15:58:58]: For more information, check out https://docs.fastlane.tools/getting-started/ios/setup/#use-a-gemfile
[15:59:00]: Generating fastlane environment output, this might take a few seconds...
✅ fastlane environment ✅
Stack

Key
Value

OS
10.12.6

Ruby
2.4.0

Bundler?
false

Git
git version 2.11.0 (Apple Git-81)

Installation Source
~/.rvm/rubies/ruby-2.4.0/bin/fastlane

Host
Mac OS X 10.12.6 (16G29)

Ruby Lib Dir
~/.rvm/rubies/ruby-2.4.0/lib

OpenSSL Version
OpenSSL 1.1.0f  25 May 2017

Is contained
false

Is homebrew
false

Is installed via Fabric.app
false

Xcode Path
/Applications/Xcode.app/Contents/Developer/

Xcode Version
8.3.3

System Locale

Variable
Value

LANG
en_AU.UTF-8
✅

LC_ALL

LANGUAGE

fastlane files:
`./fastlane/Fastfile`

fastlane_require xcodebuild

desc "Increment framework version"
lane :build_framework do |lane|

xcodebuild(
	clean:true
)

end```

**No Appfile found**

### fastlane gems

| Gem      | Version | Update-Status |
| -------- | ------- | ------------- |
| fastlane | 2.48.0  | ✅ Up-To-Date  |

### Loaded fastlane plugins:

**No plugins Loaded**

Loaded gems

| Gem                       | Version      |
| ------------------------- | ------------ |
| did_you_mean              | 1.1.2        |
| executable-hooks          | 1.3.2        |
| bundler-unload            | 1.0.2        |
| rubygems-bundler          | 1.4.4        |
| bundler                   | 1.15.3       |
| io-console                | 0.4.6        |
| slack-notifier            | 1.5.1        |
| CFPropertyList            | 2.3.5        |
| claide                    | 1.0.2        |
| colored2                  | 3.1.2        |
| nanaimo                   | 0.2.3        |
| xcodeproj                 | 1.5.1        |
| rouge                     | 2.0.7        |
| xcpretty                  | 0.2.8        |
| terminal-notifier         | 1.8.0        |
| unicode-display_width     | 1.3.0        |
| terminal-table            | 1.8.0        |
| plist                     | 3.3.0        |
| public_suffix             | 2.0.5        |
| addressable               | 2.5.1        |
| multipart-post            | 2.0.0        |
| word_wrap                 | 1.0.0        |
| tty-screen                | 0.5.0        |
| babosa                    | 1.0.2        |
| colored                   | 1.2          |
| highline                  | 1.7.8        |
| commander-fastlane        | 4.4.5        |
| excon                     | 0.57.1       |
| faraday                   | 0.12.2       |
| unf_ext                   | <IP_ADDRESS>      |
| unf                       | 0.1.4        |
| domain_name               | 0.5.20170404 |
| http-cookie               | 1.0.3        |
| faraday-cookie_jar        | 0.0.6        |
| fastimage                 | 2.1.0        |
| gh_inspector              | 1.0.3        |
| uber                      | 0.1.0        |
| declarative               | 0.0.9        |
| declarative-option        | 0.1.0        |
| representable             | 3.0.4        |
| retriable                 | 3.0.2        |
| mime-types-data           | 3.2016.0521  |
| mime-types                | 3.1          |
| little-plugger            | 1.1.4        |
| multi_json                | 1.12.1       |
| logging                   | 2.2.2        |
| jwt                       | 1.5.6        |
| memoist                   | 0.16.0       |
| os                        | 0.9.6        |
| signet                    | 0.7.3        |
| googleauth                | 0.5.3        |
| httpclient                | 2.8.3        |
| google-api-client         | 0.12.0       |
| mini_magick               | 4.5.1        |
| multi_xml                 | 0.6.0        |
| rubyzip                   | 1.2.1        |
| security                  | 0.1.3        |
| xcpretty-travis-formatter | 0.0.4        |
| dotenv                    | 2.2.1        |
| faraday_middleware        | <IP_ADDRESS>     |
| json                      | 2.1.0        |
| openssl                   | 2.0.4        |

*generated on:* **2017-07-25**

@drekka The problem here is that you need to wrap xcodebuild in quotes on line 1 of your script. So, instead of having:
fastlane_require xcodebuild

You should have:
fastlane_require "xcodebuild"

If you don't do this (and this is the interesting part!), it will actually run xcodebuild as part of the fastlane_require line! And that produces weird output like the Installing Ruby gem '▸ Building DigitalID/DigitalID-iOS [(Release)] line you saw.
I hope this helps, please let us know if you have other questions about fastlane! 🚀

ok, that was a WTF moment. Thanks, I would never have figured that out.


Change default vpc name to mantl

[x] Installs cleanly on a fresh build of most recent master branch
[x] Upgrades cleanly from the most recent release
[x] Updates documentation relevant to the changes

can we expose the long_name var in aws.sample.tf so that it can be customized? (like we do in gce.sample.tf, for example)

@ryane Is that what you had in mind?

yes, that way the vpc name can be set to something other than mantl. Another question is whether we actually need a long_name and a short_name but this lgtm for now.

verified I could customize vpc name and aws build passed.


19614 - Bantu Lala - Update lat-lon for song 2475; Change culture wiki; Suggest name change

Lat-lon for song 2475 is recording location when it should be performer location (Serenje District, Zambia;  -13.35, 30.52).
Also, culture wiki on society metadata was for Bantu people as a whole. Suggest changing to Lala people.
Suggest renaming society "Lala". Lala is a Bantu language so I don't see the need to specify that in the name.

Suggestions implemented


NoTag: reader does not contain an id3 tag

I am getting the NoTag: reader does not contain an id3 tag trying to get the ID3 of a my library. Please tell me if it's an issue for id3 or for audiotags :)
Node.js' music-metadata can parse those with no issues.
I have put a few of those files in this drive folder in case that can help.

That error comes from id3

Your files have ID3v1 tags, and audiotags assumes ID3v2. id3 has v1v2::read_from_path to handle both if you'd like to refactor audiotags to use that.

Was or Is there a reason why somebody would want to read only v2 tag and not v1v2 behind abstraction - I'm wondering why there is explicit abstraction at id3 level. Sounds like it would be preferable to assume v1v2 everywhere vs v2 as of now ?

I mean it does make sense for writing.. yeah so I guess reading for mp3 can use v1v2 by default ?

Thank you @Serial-ATA, let me try that :)

Also since Id3 doesn't support writing Id3v1 we either need to label it read-only ?

Is there possibility to lose any information in the conversion from v1 to v2 tags e.g. were any of the fields dropped and are there any incompatibility concerns otherwise e.g. where a field loses data going to v2 from v1 ?
If there are any data loss / corruption issues this sounds dicy without making this conversion explicit where the intent is clear from API perspective that it will be lossy (if any) ?
Though given there is abstraction at v1v2 at id3 already this is just an upgrade right without any loss / corruption ?

Is there possibility to lose any information in the conversion from v1 to v2 tags e.g. were any of the fields dropped and are there any incompatibility concerns otherwise e.g. where a field loses data going to v2 from v1 ?

v1 to v2 is completely lossless since v1 is so basic (only supports title, artist, album, year, comment, track number).
The only issue it would theoretically cause is someone is using an incredibly old app that doesn't understand ID3v2 tags, otherwise it's just an overall upgrade since it allows for much more future metadata expansion.


Add GitHub workflow to build exe

Ah okay. I missed the rust action. would be cool if you could have dependencies on other workflows but i agree it makes sense to not bother build if tests haven't passed. I think this is a good start and even just having the artifact built to download is a great step.


Programatically Modifying Types

I see a challenge currently in TypeScript in supporting alternative methods of OO programming.  The topic has come up in several forms before, and there isn't a clear solution.  Here is a summary of related tickets all in a similar problem space:

#2919 - Mixin language support (with toy implementation)
#4038 - Treat prototype assignments like merging type declarations
#311 - Feature Request / Proposal: Traits
#563 - Partial Classes
#4302 - Proposal: structural overloading for interfaces

Currently, the only ways of "defining" types at design time:

Interfaces - they support extension and are open ended
Classes - supports traditional OO inheritance
Abstract Classes - like Interfaces, but called Abstract Classes :wink:
Generics - Allows parameterisation of types
Unions - Allow concatenation of types
Intersections - More like a merge type actually :wink:

In JavaScript, I can support other non-Inheritance and non-OO programming paradigms, but if I decide to extend those in TypeScript, I almost certainly will lose the static typing
For example, I can create a "composition/factory" in JavaScript, that makes it easy for me to "compose" functionality:
var compose = require('compose');

var factory = compose(function () { /* init */ }, {
    foo: 'bar'
}, {
    bar: 1
});

var a = factory.create();

But if I wanted all sorts of TypeScript goodness, it essentially starts to get really complicated.  I could of sort of used Generics with Intersections, but the actual definition of the type and the implementation are decoupled and users of the compose library would have to learn some "arbitrary" syntax.
Now if there was a way, behind the scenes, to programatically merge/mixin types and modify them, I could abstract the user from the implementation and it would just work...  I would love to be able to write something like this in TypeScript:
import compose from 'compose';

class Foo {
    foo: string = 'bar';
}

interface Bar {
    bar: number;
}

let factory = compose(function () { /* init */ },
    Foo,
    Bar
);

let a = factory.create();

Another example would be Aspect Oriented Programming.  It is easy to support in JavaScript and actually the Decorators help a lot in TypeScript, but the challenge is that Decorators don't modify the type definition, and with after and around advice the return type can be modified.  There is currently no way of expressing this in TypeScript.
There are probably loads of other patterns that are implemented in JavaScript which we cannot easily use in TypeScript and currently, it means us in the community are like a load of needy children, saying "I want, I want, I want, I need, I need, I need".  Obviously a lot of this boils down to pseudo-religious debates, but without design time programmatic access to accessing and manipulating the types, we are going to constantly be running into these conflicts.
I am not sure of what the most logical and clean way the TypeScript team could implement something like this, but hopefully the team sees the need...

So... you can actually type your example fairly nicely in TS 1.6:
interface MixinFactory<T> {
	create: () => T;
}

type AnyCtor<T> = (() => void) | (new() => T);

function compose<A>(ctor: AnyCtor<A>, a: A): MixinFactory<A>;
function compose<A, B>(ctor: AnyCtor<A&B>, a: A, b: B): MixinFactory<A&B>;
function compose<A, B, C>(ctor: AnyCtor<A&B&C>, a: A, b:B, c:C): MixinFactory<A&B&C>;
function compose<A, B, C, D>(ctor: AnyCtor<A&B&C&D>, a: A, b:B, c:C, d: D): MixinFactory<A&B&C&D>;
function compose(ctor: new() => any, ...mixins: any[]): MixinFactory<any> {
	ctor.prototype = _.extend({}, ...mixins);
	return {
		create() {
			return new ctor();
		}
	};
}

The only caveat is the arbitrary number of overloads - there's no way to say that it could take an arbitrary number of type arguments and "merge them all" using current syntax.
Similarly, there's no way to get the types of an array literal passed through the system. This literal:
var myMixins = [{
	a: 2
}, {
	b: 'twenty'
}, {
	doTheThing() {
		console.log('The thing!');
	}
}];

Has the type ({a: number} | {b: string} | {doTheThing: () => void})[]. Strictly speaking, we could do better in this case if we remembered the arity of literals like this and associated each type with its slot in the tuple-like array. Considering we support tuple types in many places already, inferring this literal to be a tuple type may not be the worst idea. Giving it the type [{a: number}, {b: string}, {doTheThing: () => void}] seems way more accurate to me. If we could do that, then when I do
var factory = compose(() => {}, ...myMixins);

we know that the best overload is compose<A, B, C>, rather than the present result: the inability to find a matching overload. Unfortunately I've been informed that there's some other issues with array-vs-tuple type uncertainty which can make it difficult for us to do that (what if the array is edited? Then its type changes. How do we know it changes? Ich.).
Regardless - you can still do this:
var factory = compose(() => {

}, {
	a: 2
}, {
	b: 'twenty'
}, {
	doTheThing() {
		console.log('The thing!');
	}
});
export var __instance = factory.create();
export type MyClass = typeof __instance;
export var create = factory.create;

(I personally think that it's cute that you can extract the type in that way, though it would be better if typeof could resolve a bit more than namespaces on its own)
And have full types flowing throughout your code.

@weswigham thanks! The overloading hadn't fully occurred to me.  A couple of thoughts:

The only caveat is the arbitrary number of overloads - there's no way to say that it could take an arbitrary number of type arguments and "merge them all" using current syntax.

I think it has been discussed other places, but potentially a rest operator for generics is a part solution?  Though how to operate it it programmatically would still need to be addressed unless it is implied a ...R generic is always a intersection.  Something like this:
function compose<...R>(ctor: AnyCtor<R>, /*...*/);

Another thought, extracting the type via typeof is certainly useful.  I think the challenge, I am seeing at the moment is while the extra "hoops" to jump through to extract the class information would be ok for some writing code, it become increasingly difficult for me to create a rational API I could reasonably get someone else to use (same to the challenges of the earlier of having to decide the arbitrary number of types).
I mentioned Decorators above too, but I didn't give an example.  Here is where I see things becoming difficult:
interface AfterAdvice {
	(target: Object, result: any, args: any[]);
}

function after(advice: AfterAdvice) {
	return function(target: Object, propertyKey: string, descriptor: TypedPropertyDescriptor<any>): TypedPropertyDescriptor<any> {
		let origFn = target[propertyKey];
		return {
			value: function () {
				let result = origFn.call(target, arguments);
				return advice.call(target, result, arguments);
			}
		}
	}
}

class A{
	@after(function () {
		return true;
	})
	foo(): string {
		return 'true';
	}
}

If I want a decorator to modify the type, I can't.  I think the logic is the same for class decorators as well.  It would be great if a decorator had some sort of ability to not only install functionality but change the type as well.
I guess what I am asking for is that now is to allow operators to work on type aliases... for example:
let type MyClass = typeof instance;
MyClass.foo = typeof (a: string, b: number) => void;
/* now MyClass type/interface has a property of foo which is a function */
let myFactory = compose<MyClass>(/*...*/);

Then I can do all sort of "magic" including handling the merging of type, intersections, conflict resolution, decoration and manipulation without it having to be supported "natively" in TypeScript...

Though how to operate it it programmatically would still need to be addressed unless it is implied a ...R generic is always a intersection.

Sounds like you'd want $Either as well - check out #2710.

This really has been solved in lots of other ways.  I don't have a legitimate use case anymore for this.


acronym: fixed typos, crypto-square: upgraded scala version

acronym: fix for two typos in class and function name
crypto-square: upgraded Scala version

Thanks. Would you please upgrade the version in a separate commit/PR?

I have split this into two separate PRs.


usm: Temporarly disable grpc test

What does this PR do?
Disabling tests after https://github.com/DataDog/datadog-agent/pull/20082 change broke the tests

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] 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.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

Since the CI is broken, force merging


Update the png files.

Make sure that all the png files go-to a single folder instead of separate input.

Hi! I would like to work on it.

I am looking at the png files in the directory in my editor, I just don't understand what the issue precisely is. Do you just want all the png files in one folder? As in to make one folder in the directory for them?

Yup u got it right @elissavetv

Awesome! Doing it now :D

@elissavetv Seems like @Kameramann23 has fixed this.

Oh ok :( Is there anything else I can work on?

Oh yes @elissavetv I am gonna update the issues. Please keep a check on them and also you can try the existing issues.


Request pretrained models

I can not download the D0-D7 pth pretrained models because of the connection, who can provide them to me? Thanks .
<EMAIL_ADDRESS> /
<EMAIL_ADDRESS>

@iamastar88
check here

python train.py --dataset VOC --dataset_root data/VOCdevkit/ --network efficientdet-d4 --batch_size
12 --gpu 7
train.py:304: UserWarning: You have chosen to seed training. This will turn on the CUDNN deterministic setting, which can slow down your training considerably! You may see unexpected behavior when restarting from checkpoints.
warnings.warn('You have chosen to seed training. '
train.py:311: UserWarning: You have chosen a specific GPU. This will completely disable data parallelism.
warnings.warn('You have chosen a specific GPU. This will completely '
Use GPU: 7 for training
Downloading: "http://storage.googleapis.com/public-models/efficientnet/efficientnet-b4-6ed6700e.pth" to /root/.cache/torch/checkpoints/efficientnet-b4-6ed6700e.pth
Traceback (most recent call last):
File "train.py", line 335, in
main()
File "train.py", line 331, in main
main_worker(args.gpu, ngpus_per_node, args)
File "train.py", line 234, in main_worker
D_class=EFFICIENTDET[args.network]['D_class']
File "/asff/EfficientDet.Pytorch-master/models/efficientdet.py", line 33, in init
self.backbone = EfficientNet.from_pretrained(MODEL_MAP[network])
File "/asff/EfficientDet.Pytorch-master/models/efficientnet.py", line 243, in from_pretrained
model, model_name, load_fc=(num_classes == 1000))
File "/asff/EfficientDet.Pytorch-master/models/utils.py", line 319, in load_pretrained_weights
state_dict = model_zoo.load_url(url_map[model_name])
File "/usr/local/lib/python3.6/dist-packages/torch/hub.py", line 462, in load_state_dict_from_url
_download_url_to_file(url, cached_file, hash_prefix, progress=progress)
File "/usr/local/lib/python3.6/dist-packages/torch/hub.py", line 372, in _download_url_to_file
u = urlopen(url)
File "/usr/lib/python3.6/urllib/request.py", line 223, in urlopen
return opener.open(url, data, timeout)
File "/usr/lib/python3.6/urllib/request.py", line 532, in open
response = meth(req, response)
File "/usr/lib/python3.6/urllib/request.py", line 642, in http_response
'http', request, response, code, msg, hdrs)
File "/usr/lib/python3.6/urllib/request.py", line 570, in error
return self._call_chain(*args)
File "/usr/lib/python3.6/urllib/request.py", line 504, in _call_chain
result = func(*args)
File "/usr/lib/python3.6/urllib/request.py", line 650, in http_error_default
raise HTTPError(req.full_url, code, msg, hdrs, fp)
urllib.error.HTTPError: HTTP Error 403: Forbidden
U can download with 迅雷。+ http://storage.googleapis.com/public-models/efficientnet/efficientnet-b4-6ed6700e.pth
But I try val mAP=0 ，except D0 and D1。
good luck。

python train.py --dataset VOC --dataset_root data/VOCdevkit/ --network efficientdet-d4 --batch_size
12 --gpu 7
train.py:304: UserWarning: You have chosen to seed training. This will turn on the CUDNN deterministic setting, which can slow down your training considerably! You may see unexpected behavior when restarting from checkpoints.
warnings.warn('You have chosen to seed training. '
train.py:311: UserWarning: You have chosen a specific GPU. This will completely disable data parallelism.
warnings.warn('You have chosen a specific GPU. This will completely '
Use GPU: 7 for training
Downloading: "http://storage.googleapis.com/public-models/efficientnet/efficientnet-b4-6ed6700e.pth" to /root/.cache/torch/checkpoints/efficientnet-b4-6ed6700e.pth
Traceback (most recent call last):
File "train.py", line 335, in
main()
File "train.py", line 331, in main
main_worker(args.gpu, ngpus_per_node, args)
File "train.py", line 234, in main_worker
D_class=EFFICIENTDET[args.network]['D_class']
File "/asff/EfficientDet.Pytorch-master/models/efficientdet.py", line 33, in init
self.backbone = EfficientNet.from_pretrained(MODEL_MAP[network])
File "/asff/EfficientDet.Pytorch-master/models/efficientnet.py", line 243, in from_pretrained
model, model_name, load_fc=(num_classes == 1000))
File "/asff/EfficientDet.Pytorch-master/models/utils.py", line 319, in load_pretrained_weights
state_dict = model_zoo.load_url(url_map[model_name])
File "/usr/local/lib/python3.6/dist-packages/torch/hub.py", line 462, in load_state_dict_from_url
_download_url_to_file(url, cached_file, hash_prefix, progress=progress)
File "/usr/local/lib/python3.6/dist-packages/torch/hub.py", line 372, in _download_url_to_file
u = urlopen(url)
File "/usr/lib/python3.6/urllib/request.py", line 223, in urlopen
return opener.open(url, data, timeout)
File "/usr/lib/python3.6/urllib/request.py", line 532, in open
response = meth(req, response)
File "/usr/lib/python3.6/urllib/request.py", line 642, in http_response
'http', request, response, code, msg, hdrs)
File "/usr/lib/python3.6/urllib/request.py", line 570, in error
return self._call_chain(*args)
File "/usr/lib/python3.6/urllib/request.py", line 504, in _call_chain
result = func(*args)
File "/usr/lib/python3.6/urllib/request.py", line 650, in http_error_default
raise HTTPError(req.full_url, code, msg, hdrs, fp)
urllib.error.HTTPError: HTTP Error 403: Forbidden
U can download with 迅雷。+ http://storage.googleapis.com/public-models/efficientnet/efficientnet-b4-6ed6700e.pth
But I try val mAP=0 ，except D0 and D1。
good luck。

can you give me D0-D7 pth pretrained models ?please,thank you very much?

I can not download the D0-D7 pth pretrained models because of the connection, who can provide them to me? Thanks .
<EMAIL_ADDRESS>/
<EMAIL_ADDRESS>
Do you get those Pth? if yes,Please send me a copy?
wait for your reply,thanks!

I can not download the D0-D7 pth pretrained models because of the connection, who can provide them to me? Thanks .
<EMAIL_ADDRESS>/
<EMAIL_ADDRESS>
Do you get those Pth? if yes,Please send me a copy?
wait for your reply,thanks!

checkout my repo,
The pytorch re-implement of the official efficientdet with SOTA performance in real time and pretrained weights.
there are D0-D7 weights
https://github.com/zylo117/Yet-Another-EfficientDet-Pytorch

thank  you very  much！<PHONE_NUMBER><EMAIL_ADDRESS>签名由 网易邮箱大师 定制 On 04/11/2020 13:13, zylo117 wrote: I can not download the D0-D7 pth pretrained models because of the connection, who can provide them to me? Thanks<EMAIL_ADDRESS>/<EMAIL_ADDRESS>Do you get those Pth? if yes,Please send me a copy? wait for your reply,thanks! checkout my repo, The pytorch re-implement of the official efficientdet with SOTA performance in real time and pretrained weights. there are D0-D7 weights https://github.com/zylo117/Yet-Another-EfficientDet-Pytorch — You are receiving this because you commented. Reply to this email directly, view it on GitHub, or unsubscribe.


Retry flash card 3 times if it fails

Let's change this to constantly retry until it gets a successful response.


additions to c\dsc and c\dse

If the user has vim-surround installed, how about adding (perhaps only in the documentation) something like
augroup latexSurround
   autocmd!
   autocmd FileType tex call s:latexSurround()
augroup END

function! s:latexSurround()
   let b:surround_{char2nr("e")} = "\\begin{\1environment: \1}\n\t\r\n\\end{\1\1}"
   let b:surround_{char2nr("c")} = "\\\1command: \1{\r}"
endfunction

so that there is a corresponding ysc and yse command? Note that yse is somewhat superfluous because there is (experimentally) already ysl in vim-surround by default.

Could you explain what this does? Does it only provide ysc and yse, or does it provide any of the other vim-surround functionalitites as well?

Not much more than that. Of course, also Se/c in visual mode yss/Se/c and the likes. But nothing like dsc/e or csc/e because you can't customize targets: https://github.com/tpope/vim-surround/issues/130


Super-gradients version 3.1.3 is giving import error

🐛 Describe the bug
I am getting the following error when I am trying to install the supergradients  library in the local machine.
Any idea how to solve this ?

Versions
Collecting environment information...
PyTorch version: 2.0.1+cu117
Is debug build: False
CUDA used to build PyTorch: 11.7
ROCM used to build PyTorch: N/A
OS: Ubuntu 20.04.6 LTS (x86_64)
GCC version: (Ubuntu 9.4.0-1ubuntu1~20.04.1) 9.4.0
Clang version: Could not collect
CMake version: version 3.26.4
Libc version: glibc-2.31
Python version: 3.11.3 | packaged by conda-forge | (main, Apr  6 2023, 08:57:19) [GCC 11.3.0] (64-bit runtime)
Python platform: Linux-5.4.0-149-generic-x86_64-with-glibc2.31
Is CUDA available: True
CUDA runtime version: Could not collect
CUDA_MODULE_LOADING set to: LAZY
GPU models and configuration: GPU 0: NVIDIA GeForce RTX 4090
Nvidia driver version: 525.105.17
cuDNN version: Could not collect
HIP runtime version: N/A
MIOpen runtime version: N/A
Is XNNPACK available: True
CPU:
Architecture:                    x86_64
CPU op-mode(s):                  32-bit, 64-bit
Byte Order:                      Little Endian
Address sizes:                   46 bits physical, 48 bits virtual
CPU(s):                          16
On-line CPU(s) list:             0-15
Thread(s) per core:              2
Core(s) per socket:              8
Socket(s):                       1
NUMA node(s):                    1
Vendor ID:                       GenuineIntel
CPU family:                      6
Model:                           79
Model name:                      Intel(R) Core(TM) i7-6900K CPU @ 3.20GHz
Stepping:                        1
CPU MHz:                         2480.705
CPU max MHz:                     4000.0000
CPU min MHz:                     1200.0000
BogoMIPS:                        6396.35
Virtualization:                  VT-x
L1d cache:                       256 KiB
L1i cache:                       256 KiB
L2 cache:                        2 MiB
L3 cache:                        20 MiB
NUMA node0 CPU(s):               0-15
Vulnerability Itlb multihit:     KVM: Vulnerable
Vulnerability L1tf:              Mitigation; PTE Inversion
Vulnerability Mds:               Mitigation; Clear CPU buffers; SMT vulnerable
Vulnerability Meltdown:          Mitigation; PTI
Vulnerability Mmio stale data:   Mitigation; Clear CPU buffers; SMT vulnerable
Vulnerability Retbleed:          Not affected
Vulnerability Spec store bypass: Mitigation; Speculative Store Bypass disabled via prctl and seccomp
Vulnerability Spectre v1:        Mitigation; usercopy/swapgs barriers and __user pointer sanitization
Vulnerability Spectre v2:        Mitigation; Retpolines, IBPB conditional, IBRS_FW, STIBP conditional, RSB filling, PBRSB-eIBRS Not affected
Vulnerability Srbds:             Not affected
Vulnerability Tsx async abort:   Mitigation; Clear CPU buffers; SMT vulnerable
Flags:                           fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush dts acpi mmx fxsr sse sse2 ss ht tm pbe syscall nx pdpe1gb rdtscp lm constant_tsc arch_perfmon pebs bts rep_good nopl xtopology nonstop_tsc cpuid aperfmperf pni pclmulqdq dtes64 monitor ds_cpl vmx est tm2 ssse3 sdbg fma cx16 xtpr pdcm pcid dca sse4_1 sse4_2 x2apic movbe popcnt tsc_deadline_timer aes xsave avx f16c rdrand lahf_lm abm 3dnowprefetch cpuid_fault epb cat_l3 cdp_l3 invpcid_single pti intel_ppin ssbd ibrs ibpb stibp tpr_shadow vnmi flexpriority ept vpid ept_ad fsgsbase tsc_adjust bmi1 hle avx2 smep bmi2 erms invpcid rtm cqm rdt_a rdseed adx smap intel_pt xsaveopt cqm_llc cqm_occup_llc cqm_mbm_total cqm_mbm_local dtherm ida arat pln pts md_clear flush_l1d
Versions of relevant libraries:
[pip3] numpy==1.24.3
[pip3] pytorch-quantization==2.1.2
[pip3] torch==2.0.1
[pip3] torch-model-archiver==0.8.0
[pip3] torchaudio==2.0.2
[pip3] torchvision==0.15.2
[pip3] triton==2.0.0
[conda] numpy                     1.24.3                   pypi_0    pypi
[conda] pytorch-quantization      2.1.2                    pypi_0    pypi
[conda] torch                     2.0.1                    pypi_0    pypi
[conda] torch-model-archiver      0.8.0                    pypi_0    pypi
[conda] torchaudio                2.0.2                    pypi_0    pypi
[conda] torchvision               0.15.2                   pypi_0    pypi
[conda] triton                    2.0.0                    pypi_0    pypi

It seems you are using python 3.11. And as the error log suggests, there is no matching version for onnxruntime.
There are a few options you can do now to get it working:

If possible, use python 3.9 or 3.10
If not - you can install SG without deps: pip install --no-deps super-gradients==3.1.3, but you will have to install rest of dependencies by hand (For onnxruntime you can skip version specifier)

@sadra-barikbin your issue is not related to the topic's starter issue.
Please keep in mind that master branch is not meant to be stable and may break at any time.
@PrajwalCogniac we haven't been heard from you for two weeks regarding this issue so I assume downgrading python helped and therefore closing the issue.


startControlPlane assumes kubebuilder etcd?

What happened:
Trying to use kuttl with startControlPlane: true and getting the following:
$ kubectl kuttl test
=== RUN   kuttl
    harness.go:462: starting setup
    harness.go:246: running tests with a mocked control plane (kube-apiserver and etcd).
    harness.go:513: cleaning up
    harness.go:570: removing temp folder: ""
    harness.go:595: fatal error getting client: unable to start control plane itself: failed to start the controlplane. retried 5 times: fork/exec /usr/local/kubebuilder/bin/etcd: no such file or directory
--- FAIL: kuttl (0.00s)
FAIL

Is there something that is hard-coded looking for etcd at that kubebuilder path?

I hacked up a work-around, which is to run everything from within a container environment, https://github.com/apache/hbase-kustomize/tree/main/dockerfiles/kuttl

Is there something that is hard-coded looking for etcd at that kubebuilder path?

I'm no expert on this, but I think it's customizable. Admittedly this should be better documented here. This might be a good place to start.


Hard to get started contributing

I don't use pnpm, so getting started checking out and building this project was a bit harder for me than other standard npm projects.
npm ci failed, which at least let me know that I needed to read instructions and see what package manage this project used. I didn't see pnpm installation instructions, so I tried npm i -g pnpm, then pnpm install and pnpm start as instructed, but that just gave me this error message:
 ERR_PNPM_BAD_PM_VERSION  This project is configured to use v9.0.5 of pnpm. Your current pnpm is v9.0.6

If you want to bypass this version check, you can set the "package-manager-strict" configuration to "false" or set the "COREPACK_ENABLE_STRICT" environment variable to "0"

Is there a reason this project is configured to use a specific patch version of pnpm? I don't know where the "package-manager-strict" setting is, so I have to go read pnpm docs now...
Is there an important reason to use pnpm? This isn't an extremely large monorepo. Is the performance difference that great? Initial install took 6.6s on my machine, which doesn't feel much different than what npm would have taken.

This is a two project monorepo, so if there is a way to configure npm to do the same, i can be for that (without getting in to a package manager fight 😅).
Here are the requirements though:

root dir should be the package
tests-public is its own nested package with separate tests so that we can verify the package.json#exports is correct, and signal-utils builds successfully (these tests will fail if signal-utils is not built, which is fine as they aren't the primary tests)

The nice to haves, and why a default to pnpm
(Tldr: habit)

0 disk space install (basically, because i already have the deps in my main cache elsewhere on disk, so it's all links)
easier time fixing external dependencies who define their package.json wrong (common, but irrelevant for this repo)
i use it for all my other projects

I do understand there is value in 'the defaults',
But, it seems corepack needs some work, because it seems silly to error, and that automatically installing the configured package manager and specific version ... For this project, it probably doesn't matter, but specific versions do matter on bigger projects. I'd want to steer forks away from corepack! Or at least manual tooling management as corepack seems like a partial tool so far, i'm sure they're working on it tho. Automating all that is one of my favorite productivity hacks in OSS! (I use volta atm (not a requirement for contribution), but i'm open to switching to other stuff, this space is quite big!)
At the very least the packagemanager field is package.json should probs be just the major.

I don't know where the "package-manager-strict" setting is

Smells like an .npmrc setting

Yeah, I don't mean to start a package manager fight. It might just be that non-pnpm users don't know how to get started and so need just a little guidance.

At the very least the packagemanager field is package.json should probs be just the major.

This (and a link  or inline instructions on installing pnpm) sounds like it might fix most of the friction?

fwiw, how I'd probably set this project up
npm workspaces have worked well for me, although it does hoist be default which can lead to bad imports if you're not careful. I tend to use linting for that while I wait for install-strategy=linked to work for me.
I'm pretty sure that the top-level package being the main package works, even though my monorepos don't have a main package. You just like the nested package in the workspaces setting.
So I would do that, and instead of Vite and Babel, I would use plain TypeScript, node:test, and then Wireit to make sure that the package is built before the tests. I know Wireit is a non-standard tool, and an opinion, but it lets you continue to use standard npm commands while coordinating scripts.
I won't suggest you actually do this :) I'm willing to get things setup on my end to contribute!

I've opened https://github.com/proposal-signals/signal-utils/pull/58 for feedback you have the time! would be super appreciated :tada:

Gonna close for now, as the readme is updated, if there is we're discussion to have, lemme know and i can reopen


fix for bad merge error

@bdash do you know why this is causing test failures? Not even sure which is the correct change. Trying to fix linux compilation errors. The test failures are as follows:
-------------------------------------------------------------------------------
list
  add_notification_block()
  multiple LinkViws for the same LinkList can get notifications
-------------------------------------------------------------------------------
/Users/ari/src/realm/realm-js/src/object-store/tests/list.cpp:39
...............................................................................

/Users/ari/src/realm/realm-js/src/object-store/tests/list.cpp:212: FAILED:
due to unexpected exception with message:
  std::exception

-------------------------------------------------------------------------------
list
  add_notification_block()
  tables-of-interest are tracked properly for multiple source versions
-------------------------------------------------------------------------------
/Users/ari/src/realm/realm-js/src/object-store/tests/list.cpp:39
...............................................................................

/Users/ari/src/realm/realm-js/src/object-store/tests/list.cpp:310: FAILED:
due to unexpected exception with message:
  std::exception

===============================================================================
test cases:   47 |   46 passed | 1 failed
assertions: 5392 | 5390 passed | 2 failed

You removed the call to transaction::advance(m_sg, *m_current, version);.

Thanks. That would definitely explain things....


More glibc fixes

Trying to get more linux builds updated to use older debian and older glibc requirement.

Defunct; partially fixed by #154. Still need build setup for i386 and exotic platforms.


[BUG] Simple typo (en)

你好
谢谢你的这个项目！
Nǐ hǎo
xièxiè nǐ de zhège xiàngmù!
Describe the bug 描述你遇到的错误
Simple (english) typo (misspelling): discription should be: description
I'm sorry but I don't really understand how to use GIT properly yet, or I would have contributed the fix.
I just did a simple Search/Replace with a text editor on the entire theme/hugo site.
A clear and concise description of what the bug is.
Feature Request: Is there an easy way to make the pages PRINT (on paper) well? Right now, I get the top navigation bar repeating itself and blocking page content (that should be printed) behind it. I find this is a problem on a LOT of Hugo-generated sites. I understand that it is not a Hugo-specific problem, but more a CSS issue.
You may see this yourself if you try to print out a page, such as the theme-documentation-extended-shortcodes/ page.

Thanks for your bug reports! @darrenaddy


[New Process Improvement Need]: Target architectural guides for making software more generalized and multi-mission

Checked for duplicates
Yes - I've already checked
Category
Information Sharing - documentation design, templates, communication, etc.
Describe the need
We have a need for targeted architectural guides for making software more generalized and multi-mission. (+1'd by @AaronPlave)

+1'd by @ramesh-maddegoda (Unity, PDS)


Bug: Log reports error reading properties, Home.app then reports lock "was jammed"

Describe The Bug
Home.aLog reports error reading properties, Home.app then sends notification lock "was jammed"pp reports lock "was jammed" despite lock not being jammed. August app and other plugins do not report jam.
To Reproduce
Error is random with no seeming correlation to time since lock or unlock.
Expected behavior
No error, no Home.app notification.
Relevant log output
[10/8/2022, 6:26:09 PM] [August] TypeError: Cannot read properties of undefined (reading 'retryCount')
[10/8/2022, 6:26:09 PM] [August] Lock: Front Door failed lockStatus (refreshStatus), Error Message: "Cannot read properties of undefined (reading 'retryCount')"

Config for homebridge-august
"name": "August",
            "credentials": {
                "augustId"<EMAIL_ADDRESS>                "password": "xxxxxx",
                "validateCode": "xxxxx",
                "notice": "Keep your tokens a secret!"
            },
            "options": {
                "logging": "standard"
            },
            "platform": "August"

Screenshots

Device & Model
AugusAugust WiFi Smart Lock
Node.js Version
16.17.1
NPM Version
8.5.0
Homebridge Version
oznu/homebridge:latest
Homebridge August Plugin Version
4.50.1
Homebridge Config UI X Plugin Version
1.0.4
Operating System
Docker

@hrhnick, currently if I get a retry count I report jammed. I think I will remove that code.

Seeing the same error.

@donavanbecker could this somehow trigger the device to try unlock and relock? (To clear the jam?)
I can’t be sure but I am pretty certain I was seeing some odd unlocks as well but couldn’t find it in the logs.

@hrhnick @3ne, lets try v1.0.5

@hrhnick @3ne, lets try v1.0.5

Updating.

From my experience, the update has cleaned up the jam alerts, and I'm getting normal lock/unlock alerts. I was having the same issues OP was.

JAMMED should only happen if the API sends a false for both Locked and Unlocked.

@donavanbecker all looks good ever here as well.

Good to hear!


Add action outputs for site id and url

This PR adds support for github action outputs.
I think this is a decent workaround for sites that have more complex requirements to make use of other github actions in a workflow / pipeline.
Closes #31

Great idea, thanks!


Figured out what vm32x and related are

There's a note in Codegen.hs about how I didn't know what vm32x vm32y vm64x vm64y are.
They're 32 and 64-bit memory operands. We can finally support vgatherpd. Woo.

I'm hesitant to fix this because it's going to introduce new values into the opcode table and force a big fix in stoke. Let's wait on this until we need it or until things are less in flux.


[5.6] Feature/refactor Str::languageSpecificCharsArray()

Description
Improve readability of Support\Str::languageSpecificCharsArray()
Motivation and context
Associative array could be easier read and support as to different arrays, because you can always see which value refers to which value
How has this been tested?
All tests passed successfully
Types of changes

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

Checklist:

[x] I have read the CONTRIBUTING document.
[x] My pull request addresses exactly one patch/feature.
[x] I have created a branch for this patch/feature.
[x] Each individual commit in the pull request is meaningful.
[x] I have run tests to cover my changes.

I understand what you're trying to do, but this part of the code is taken from the Stringy library. Your changes will be lost next time we update this method.
https://github.com/danielstjules/Stringy/blob/df24ab62d2d8213bbbe88cc36fc35a4503b4bd7e/src/Stringy.php#L1869-L1878

@sisve hm, good notice :+1: Should I contribute then to danielstjules/stringy?

Pull Request to original repository in progress https://github.com/danielstjules/Stringy/pull/179


schemas: duration_type -> phase, +duration_days +context

start_date -> date (in visit)
Brings this in line with date in observations
duration_days (int)
Refers to the time period in which observation happened.
Reference for duration_days is date
context (str)
Refers to the context of an observation. For example, if
the observation is temperature, could refer to axillary
or ear, or if the observation is oxygen saturation, could
refer to whether the measurement happened in room air
or post oxygen therapy.

looks ok. is there support in the schema for calculating duration_days? assuming I got this right ..


Flares and Torches don't start fires

Description
As title.
Similar issue we've had in the past with lots of things. Matches ignite fires now and lighters use a welder component to achieve this.
Whether we rewrite the whole thing to have a consistent component or not I'll leave that to you.

Shouldn't this be closed? Looks like issue should be linked to PR #13555 with fix or close tag.

PR https://github.com/space-wizards/space-station-14/pull/13555 fixes this. This can be closed.


Android Basics: Viewmodel and State in Compose

Next, similar to the updates for the score, you need to update the word count.

Add another variable for the count in the GameUiState. Call it currentWordCount and initialize it to 1.

data class GameUiState(
   val currentScrambledWord: String = "",
   val currentWordCount: Int = 0,
   val score: Int = 1,
   val isGuessedWordWrong: Boolean = false,
)

link to hightlight
initialize it to 1 but the wrong value is initialized to 1 ;)

Hello @koraysels,
Thank you for reaching out to us and reporting this typo. The code snippet is updated:


pxlrbt\FilamentSpotlight\Actions\RegisterUserMenu::getUrl(): Return value must be of type string, null returned

pxlrbt\FilamentSpotlight\Actions\RegisterUserMenu::getUrl(): Return value must be of type string, null returned

Hello, i have the same issue and i am with the latest version of the plugin. Do i need to install another version or what?

Same issue, not sure how to move forward

Btw, my issue I found the cause by going into the vendor file RegisterUserMenu.php and adding a try catch TypeError, then I was able to find the menu item causing the plugin to crash
public static function boot(Panel $panel)
{
$self = new static;
/**
* @var array $items
*/
$items = $panel->getUserMenuItems();
    foreach ($items as $key => $item) {
        $name = $self->getName($key, $item);
        try{
            $url = $self->getUrl($key, $item);
        }catch (\TypeError $e){
            dd($item);
            $url = '';
        }

then I was able to find the menu item causing the plugin to crash

Is it a default one and a use case that should be supported? We could probably filter out results with an empty url


Document CommonQueryParameters

Documented all the parameters defined in CommonQueryParameters and CommonCatQueryParameters.
I also found source_query_string which isn't an Elasticsearch parameter but is instead likely inherited from NEST and would've been translated into ?source=... as a means of smuggling small request bodies for GET into the query string. I've removed this parameter in this PR.

Got confirmation from @Mpdreamz about source_query_string, merging now!


Enhancement: add optional parameter to specify a folder in the app catalog for m365 spo app add

Currently, SPFx solutions are deployed at the root of an app catalog, whether it's a tenant or a site collection one.
The idea is to add a new command option to specify a folder because in some organizations, there's only one tenant for the whole company, including the subsidiaries. In that case, permissions can be setup to not expose an app to everyone but only one subsidiary.
Property description idea:

Option
Description

--catalogFolderPath [catalogFolderPath]
Relative path in the tenant or site collection app catalog to add the package file. If not provided, the package file will be added to the root folder of the app catalog.

That's a great feature request. Thanks for submitting this @michaelmaillot!
Looks good to go for me. @pnp/cli-for-microsoft-365-maintainers any further remarks before opening it up?

Nice suggestion. Maybe for consistency, we should call it appCatalogFolderUrl.
Let's also check if we need something similar for other app commands like spo app get.

Before we do, let's double check if using folders in the app catalog is a supported scenario and we're not trying to promote something that's officially shouldn't be done (even though it's working).

@waldekmastykarz - that's a really good question. We have not really considered the scenario at this level, but technically that should work as file is still in the catalog and this will provide proper level of grouping between the solutions if there are a lot of solutions. Let me run this through with few people internally to avoid surprises.

@VesaJuvonen did you had any chance to recheck this case?


Paper -> CoT pipeline: Optimize prompt for CoT extraction from paper

Requirements:

Optimize the TASK section (instructions on how to extract the question/chain/answer set)
Optimize the CHAIN OF REASONING CRITERIA section (criteria for choosing the best question/chain/answer set from a paper)

Deliverable: the final prompt

Here is the full archive of artifacts (inference data for profiling and CoT extraction, JSONL training files) for a test run of 100 papers through the pipeline:
paper-cot-extraction-test-data.tar.gz
The CoT extraction logs are in results/inference with -cot-extraction.txt extension.


I change Add report to +Add new report and plus symbol should be an icon

Targets #{ISSUE_ID}

Description

i change Add report to + Add New Report  and plus symbol be an icon

Checklist:

[x] I have performed a self-review of my own code.

@Amanraj1315 Please add the ISSUE_ID in the description.