Fix high risk world type name, remove from PVP world types

This world type is not specifically a PVP world type; at the time of
this commit, world 365 is a non-PVP high-risk world, where the world is
the same as a regular members' world, except that the Protect Item
prayer is disabled.
Fixes #8605
Fixes #8624

I think this breaks the "next" hop keyboard shortcut when hopping in the wilderness.

https://github.com/xdesr/runelite/commit/aa736cc8998da671a498d7ad3cc08ac6ef490ca0
I imagine that would not break the next shortcut but the js at: https://api.runelite.net/runelite-1.5.21-SNAPSHOT/worlds.js needs to be updated. i'll read my own json file and do some testing.

the http-service worlds endpoint is basically mirroring the Jagex one. In it, high risk worlds are a thing so it shouldn't be removed from ours. It is not relevant to whether or not the world hopper respects it any.

I'm thoroughly confused about what you mean by removing it from the http api. It's not that it needs to be removed, it's that it is poorly named and should be renamed.
Separately, it should be removed from api.WorldType#isPvpWorld (which is not used by the world hopper plugin), because high risk worlds are not PVP worlds.

I can confirm by testing that the world hopper plugin on 98984c9 behaves the same as release.

Often pkers will use the quick hop up/down feature of the world hopper to cycle through worlds, we probably should not let them hop into high risk worlds like that.


Add PoC for exporter implementation

Problem:
As a maintainer of NGF and NIC
I want an exporter library to enable me to send data to a centralized telemetry collector,
So that I can aggregate and visualize telemetry data to make better product decisions based on how users are using the products.
Solution:
Note: this is a PoC of the design.

Add Exporter type Exporter the data to an OpenTelemetry endpoint.
The passed exported data must implement Exportable interface.
Add generation tools to generate the implementation of the Exportable interface for the data and generate the scheme.
Define common telemetry data points to be used across projects.
Add a go code example along with the instructions for an e2e demo.

Fixes https://github.com/nginxinc/nginx-gateway-fabric/issues/1318
Review notes
I'd like to get feedback on the design/approach and on the defined interfaces:

Exporter
Exportable
The approach using code generation and scheme generation.
Structure of packages

Please note that the implementation details along with the units tests, especially of the code and scheme generation part, are written with "write as fast as you can to prove it works" mindset.

closing this as it is no longer relevant


O8S trigger storage of  fake/true indicator for recall later.

Would it be possible to store the previous value for true/false for the Blizzard/Thunder indicators for recall later as an enhancement to the trigger?
Storing a variable that is cleared on recall of the Thunder. I notice you have some suppression logic already I don't know if it would be easily doable to store the value to recall later. On recall, the trigger could say something like:
(true, false)
out thunder, in blizzard
or something to the effect. I'm not really sure about the scope of the variables and if you could easily reference the timeline file to tell you if you are at the recall stage or if some other way would be ideal.
Thank you!

Triggers can add/set variables on the |data| object, and it will persist until combat ends, so that other triggers can continue to read/modify it.
The O4S triggers have a few examples of this.

Thank you. I am looking at that now and this is really helpful and very well should be able to figure it out

I'm a little confused what you're asking about.  It individually says "Fake Thunder" and "Fake Ice: Get Out", and then on Mana Release, it repeats both of these.  Did you want them on the same line?  Are you saying you're not seeing the repeat?

Yeah, I'm seeing the repeat just fine. I use the triggers in discord via TTS.  I was trying to make something more like this :
tts: function(data) {
if (data.manaCharge == 1 || data.manaCharge == 3)
if (data.stackFire == true)
return 'stack up';
if (data.stackFire == false)
return 'spread out';
if (data.manaCharge == 2 || data.manaCharge == 4) {
if (data.fakeBlizzard == true && data.fakeThunder == true && data.donutBlizzard == true)
return  'in bliz on cross';
Sorry if this is a little weird, just trying to figure it out on my own, but I don't seem to be triggering the manaCharge counter variable for the TTS.

Hmm! What do your manaCharge triggers look like?

Hey,
I got it somewhat working today actually
Mana Charge looks like this:
{
id: "O8S Mana Charge",
regex: /Kefka starts using Mana Charge/,
infoText: function(data) {
if (data.manaCharge == 1 || data.manaCharge == 3)
return 'stores next 1 fire';
if (data.manaCharge == 2 || data.manaCharge ==4)
return 'stores next 2 Thun/Bliz';
return 'Storing'
},
run: function (data) { data.manaCharge = (data.manaCharge || 0) + 1; },
},
I basically dont think it needs any infoText but I wanted to track it working or not originally.
I need to further suppress the duplicate callouts to clean it up more and figure out the correct callouts for Flagrant Fire but I'll see if I can figure that out too. After already clearing the fight it's probably not at all necessary but I still think it could be worth it if it was cleaned up.


Babel + Webpack Integration

Hello,
First of all, awesome work you've done, but i'm trying to integrate your components with react-starter-kit and i'm getting the following error:
/Users/<user>/Projects/<project>/node_modules/react-foundation-components/lib/top-bar/_styles.scss:1
(function (exports, require, module, __filename, __dirname) { @import '../common';
                                                              ^
SyntaxError: Invalid or unexpected token

Did you want to use library in CSS Modules mode and if so is sass-loader setup in your webpack config?
If you don't want to use CSS Modules and have CSS globally available, import components from 'react-foundation-components/lib/global' instead.
example
import TopBar from 'react-foundation-components/lib/global/top-bar'

I'm having the same problem with this in my webpack config:
const config = {   module: {     rules: [{         test: /\.scss$/,         loaders: [           'style-loader',           'css-loader?modules&importLoaders=2&localIdentName=[local]___[hash:base64:5]',           'sass-loader',           'postcss-loader?parser=postcss-scss'         ]     }   } };
I was able to use global syntax above as a workaround at least.

Ah never mind, I tried to do
import { Callout } from 'react-foundation-components';
which causes the error, but if I do
import Callout from 'react-foundation-components/lib/Callout';
then the error goes away, while using the above configuration.


Create uniminuto.txt

Corporación universitaria  Minuto de Dios
Corporación universitaria  Minuto de Dios

@Hannibal1022 I am sorry, the domain uniminuto.edu.co is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


fix: mute esbuild while parsing for NPM modules

Parsing NPM modules might fail. We already have a user-facing log line for that, so we don't need ESBuild's output on top of it. This PR mutes esbuild, and directs the generated warnings and errors to system logs.

Wanted to get this out today. I can still write a test for this tomorrow, if we think that's necessary

Verified locally that this does mute the ESBuild errors. Updated the error message to not refer to the esbuild output.


Add options to skip release preparation and deployment.

What does this PR do?
Title.

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2023-06-13T09:22:59.262+0000

Duration: 14 min 31 sec

:grey_exclamation: Flaky test report
No test was executed to be analysed.
:robot: GitHub comments
Expand to view the GitHub comments

To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.

run benchmark tests : Run the benchmark tests.

run jdk compatibility tests : Run the JDK Compatibility tests.

run integration tests : Run the Agent Integration tests.

run end-to-end tests : Run the APM-ITs.

run windows tests : Build & tests on windows.

run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)


Access directive's ElementRef in component

Hi, is there a way that I can pass the ElementRef back to my component's class through the @Output() "inview"?
What I'm trying to do is to call a function and reference to the component that has the directive.
(inview)="isElementInViewport($event)"

It can be done using Template only

<div #ref in-view (inview)="isElementInViewport($event, ref)" ></div>


Add versioned links to docs and examples for v0.9.0

Changes

Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[ ] Includes tests (if functionality changed/added)
[ ] Includes docs (if user facing)
[ ] Commit messages follow commit message best practices

See the contribution guide for more details.
Double check this list of stuff that's easy to miss:

If you are adding a new binary/image to the cmd dir, please update
the release Task to build and release this image.

Reviewer Notes
If API changes are included, additive changes must be approved by at least two OWNERS and backwards incompatible changes must be approved by more than 50% of the OWNERS, and they must first be added in a backwards compatible way.
Release Notes
Describe any user facing changes here, or delete this block.

Examples of user facing changes:
- API changes
- Bug fixes
- Any changes in behavior

I was hoping to merge this first and do the release right afterwards.
That means that for a few minutes those links will be broken, but in future, if someone checks out the tag, the link to the docs is there in the README.

@afrittoli make sense :angel:
/hold cancel

I think this was worth a shot but i just got pinged by someone who tried to follow the docs link for 0.9.0 and was confused why it was broken, so I think in the future it might be better to merge this after the release and tag are created?

Or make sure we do it simultaneously! (With a Tekton Task maybe? :D)

I think this was worth a shot but i just got pinged by someone who tried to follow the docs link for 0.9.0 and was confused why it was broken, so I think in the future it might be better to merge this after the release and tag are created?

Waw some people are quite quick then :thinking: To @afrittoli's point, "That means that for a few minutes those links will be broken, but in future, if someone checks out the tag, the link to the docs is there in the README.", I think this is worth merging before the release (aka having this commit in the release). Maybe we need to "automate" the commit in the release pipeline :angel:


Ensure Run always returns a result, on both success and failure

https://github.com/myntra/pipeline/issues/2

Thanks for the change! This would require corresponding changes in test and example. I am accepting the PR and would make the necessary changes in the next commit.

Thanks Adnaan!


[Hub Generated] Review request for Microsoft.Maintenance to add version preview/2020-07-01-preview

This is a PR generated at OpenAPI Hub. You can view your work branch via this link.
Contribution checklist:

[ ] I have reviewed the documentation for AME onboarding.
[x] Validation tools have been run on swagger spec(s) updated and errors have all been fixed with this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.

adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Please follow the link to find more details on PR review process.

/azp run automation - sdk

Can one of the admins verify this patch?

Please go through the contribution checklist added as part of the review and mark them completed.

From the changes you are removing the type in this version applyUpdates.
Please share the details about what is the reasoning for this. Will you be deprecating this flow?


docs(alert): correct link to controller

fixes ionic-team/ionic-docs#254

Something weirder is going on here... 🤔 Closing this.


Cannot catch error when LOGIN action fails in redux/modules/auth.js login function

I've taken this example project to be the base for client side code and implemented my own API.
When calling login mehtod from redux/modules/auth.js and server responds with any HTTP error code (e.g. 401/422/500), then the promise returned is always resolved, never reaches the catch method. Although, ApiClient expectedly rejects the promise, when the API responds with an error.
this.props.login(email, password).then(() => {
  console.log('success');
}).catch(() => {
  console.log('fail');
});

I had the exact same problem now: I don't want to reset my form if the promise failed. I found out that clientMiddleware.js is not re-throwing the error after catching it.

ApiClient.js
         request.withCredentials().end(function(err, res) {
           if ( err ) {
-            if ( res.statusCode === 401) {
-              resolve(res.body);
-            }
-            reject(res.body);
+            resolve({ status: 'error', data: res.body, error: err });
           } else {
-            resolve(res.body);
+            resolve({ status: 'success', data: res.body });
           }
         });
       }).catch(function(err) {

clientMiddleware.js
      const actionPromise = promise(client);
       actionPromise.then(
-        (result) => next({...rest, result, type: SUCCESS}),
-        (error) => next({...rest, error, type: FAILURE})
+        (result) => {
+          if ( result.error ) {
+            next({...rest, error: result.error, type: FAILURE});
+          } else {
+            next({...rest, result: result.data, type: SUCCESS});
+          }
+        },
+        (error) => {
+          console.log('actionPromise error');
+          console.log(error);
+          next({...rest, error, type: FAILURE});
+        }
       ).catch((error)=> {
         console.error('MIDDLEWARE ERROR:', error);
         next({...rest, error, type: FAILURE});

I'm aware the repository isn't maintained, by making these changes I can at least hit the Redux FAIL actions by emulating the clientMiddleware respecting the ApiClient promise reject.


tf2 listener cpp tutorial: minor syntax/wording improvements

some changes i think might improve readability, as discovered in https://github.com/osrf/ros2_test_cases/issues/1176.
i would have liked to add a link to a more generic "tf concepts" page, but i didnt find anything satisfactory. i really like tho the first two paragraphs in #4235 which i found very insightful especially around the question of target/source terminology and frame order.

i would have liked to add a link to a more generic "tf concepts" page, but i didnt find anything satisfactory. i really like tho the first two paragraphs in #4235 which i found very insightful especially around the question of target/source terminology and frame order.

We have http://docs.ros.org/en/rolling/Concepts/Intermediate/About-Tf2.html , though I admit it is pretty light on details.  Still, I think it would make sense to link there, and then possibly expand that later.  What do you think?

i would have liked to add a link to a more generic "tf concepts" page, but i didnt find anything satisfactory. i really like tho the first two paragraphs in #4235 which i found very insightful especially around the question of target/source terminology and frame order.

We have http://docs.ros.org/en/rolling/Concepts/Intermediate/About-Tf2.html , though I admit it is pretty light on details. Still, I think it would make sense to link there, and then possibly expand that later. What do you think?

sounds good to me, i added a link to that page!


Files crashes when loading a Recycle Bin with many items

Description
Files crashes when loading a Recycle Bin with many items.
Steps To Reproduce
Open the Recycle Bin with around 500 items in it.
Expected behavior
Shouldn't crash.
Files Version
Version: <IP_ADDRESS> OS Version: 10.0.19044.1348
Windows Version
Version: <IP_ADDRESS> OS Version: 10.0.19044.1348
Relevant Assets
debug_fulltrust.log
debug.log

Were some files being deleted/restored in the meantime?

Nope.

Does it happen all the times?

Nah, seems to happen only after a fresh restart of the system. Perhaps even the App.

@R3voA3 is this still an issue in the preview version? https://files.community/download/preview


Remove session from nav for auth

Changes

Removed sessions from nav as the concept doesn't exist in auth

Ah the central CTA is of course off centre now because we have even amounts of nav items instead of odd now.
I think probably best to to just leave a spacer element there or give the chains list icon double spacing until we have some useful replacement 👍

Ah the central CTA is of course off centre now because we have even amounts of nav items instead of odd now.
I think probably best to to just leave a spacer element there or give the chains list icon double spacing until we have some useful replacement +1

I

Ah the central CTA is of course off centre now because we have even amounts of nav items instead of odd now.
I think probably best to to just leave a spacer element there or give the chains list icon double spacing until we have some useful replacement +1

Looks weird with a spacer or giving the chains 2 spaces and centering that

Looks weird with a spacer or giving the chains 2 spaces and centering that

Also true. @IljaDaderko any quick solutions you're aware of? Not worth bikeshedding any more on this detail in my opinion though so let's just go for it for now if not 👍

Looks weird with a spacer or giving the chains 2 spaces and centering that

Also true. @IljaDaderko any quick solutions you're aware of? Not worth bikeshedding any more on this detail in my opinion though so let's just go for it for now if not 👍

Mmm, I guess we can move WC button to the right and rest of the menu to the left, so it doesn't require even amount of items on each side. Kinda like in google's material design with that "fab" button


Refactor data step, inference step, Jupyter notebooks

This PR covers multiple "feature" changes.

refactor data step: replace CLI, rewrite some library internals, add more documentation

See #85 for further details on work completed here.

refactor train step: replace CLI, refactor & document various data processes
add new infer step: load prepped data (low-res ocean velocities) and pre-trained model, predict forcings using model
update Jupyter notebooks to use new interfaces (code de-duplication, documenting)

Extra:

document parts of the model, training (mirroring some explanations from the paper)

Important to-dos:

[x] Inference mode: Explain what input dataset must look like: velocity co-ord names
[x] Document how to use new inference mode: download pre-trained model from HuggingFace, load in dataset, run. Link from readme
[x] finish last checked-over Jupyter notebook (test_global_control)

This notebook needs two datasets:

Computed forcings from the CM2.6 dataset which are then coarsened
Output of inference (testing) step

These are then compared
~[ ] Provide option to use the ML flow style, or to load from file (e.g., xarray.open_zarr on the forcings and output of inference step).~

Pre-merge to-dos:

[x] tag current commit before merging -- perhaps mlflow-xyz? (sweeping changes here)

Fixes #87 .
Known bugs:

#104
when training, IndexError: index x is out of bounds for axis 0 with size y where x > y

somehow, the dataloaders are being asked for samples past their end?
perhaps the custom indexing over xarray is problematic for short arrays?
perhaps I broke the indexing, and it's using a coord for index? (the size is correct at least, but the index seems ~0-310)

data: unexpected OOMs

python src/gz21_ocean_momentum/cli/data.py --config-file examples/cli-configs/data-paper.yaml
with --ntimes 100, I fail on CSD3, and on my ~30GB RAM laptop

train: sometimes I get a PyTorch Runtime Error: input type (double) and bias type (float) should be the same. But I'm not sure what it's triggered by, because data types seem fine.

I can request the the module in the cli directory have their _cli_desc put as the first definition after imports with a comment that says # Description of this module or something as they provide a good explanation of what the file is about.

The section marked ## Data on HuggingFace probably needs to say something about what kind of data and trained model (i.e., that the moment it is low res) and we can update this later if we can get the high-res.


Files not from root

Hello, I want after generate css js images files, src not from root, not src='/bundle.js', but just like that src='bundle.js' , how can I do it?

Hi, can you explain more? Are you talking about static generated app ?

Yes, I talk about static generated app

@demyan1411  be my guess ...
Do you want deploy generate file to a sub folder on web host ?
if I guess correct , you need setup router.base in nuxt.config.js
module.exports = {
  router: {
    base: '/app/'
  }
}

for doc reference
https://nuxtjs.org/api/configuration-router/
it you don't have change setting , nuxt will generate all js and asset to _nuxt folder
if you want change folder name you can use build.publicPath
module.exports = {
  build: {
    publicPath: 'https://cdn.nuxtjs.org'
  }
}

for doc reference
https://nuxtjs.org/api/configuration-build/#publicpath
hope this two information can help you .

Thanks!!


Update dependencies to bullseye

When installing the deb on Raspberry Pi OS bullseye armv7l, it shows:
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
Note, selecting 'caprine' instead of './caprine_2.54.1_armv7l.deb'
Some packages could not be installed. This may mean that you have
requested an impossible situation or if you are using the unstable
distribution that some required packages have not yet been created
or been moved out of Incoming.
The following information may help to resolve the situation:

The following packages have unmet dependencies:
 caprine : Depends: libappindicator3-1 but it is not installable
E: Unable to correct problems, you have held broken packages.

After some research I found libayatana-appindicator is replacing libappindicator3-1 in bullseye. See here.
EDIT: Both AppImage and manually build with npm install failed with error similar to this:
../../sandbox/linux/seccomp-bpf-helpers/sigsys_handlers.cc:**CRASHING**:seccomp-bpf failure in syscall 0403

I reflashed my pi and the problem solved. I don't know why...
For the sandbox error, I fixed it by adding --no-sandbox or --disable-seccomp-filter-sandbox flag when launching caprine.
I will close this issue now.


Better Native Pointer Debug Views

Is your feature request related to a problem? Please describe.
I am trying to implement native bindings to a library that extensively uses pointers to objects. I am trying to build custom views using the DebuggerDisplay and DebuggerTypeProxy attributes; however, no matter what I do, I cannot get the vscode debugger views to display the objects how I want. I have tried many combinations but to no avail. The debugger views just do not like native pointers at all.
Describe the solution you would like
I have run into a few issues with the debug views that seem unsatisfactory:

The debugger view does not display the type of a pointer type at all (ex: Foo*). I have also noticed that many 'special' characters specified in the Type property of the DebuggerDisplayAttribute cause the type name for the variable to disappear in the same way: * being one of them. I think the type name should be displayed regardless of what characters the type string contains. I tested Foo* with visual studio and it correctly displays the type name
For a pointer variable in the debug view (ex: Foo*) expanding the variable reveals a dereference operation. For what I am doing, this makes absolutely no sense and I would not like the view trying to dereference the pointer and show me the result. The pointers in question are a combination of some base structure fields and opaque pointers. Provide some sort of attribute or option to tell the debug view to not try and display a dereferenced pointer.
If we try to apply the DebuggerTypeProxyAttribute to a pointer type at the assembly level, we can provide a custom view for the pointer by dereferencing the pointer properly for the application and providing the debug fields. Problem is that when you apply the DebuggerTypeProxyAttribute to a pointer type wherein the type itself is a structure, the view tries to provide a dereferenced 'raw view' of the pointer. Again, I don't want the view dereferencing the pointer and or trying to provide a raw view. Provide an option to completely disable the raw view display.
I potentially want a custom type (that is not a pointer) to be viewable through the "view binary data" button so we can view the raw memory for the variable. I have tried almost everything to try and get that button to appear for my custom type to no avail. The button appears for pointer types and just about every other type, but there seems to be no way to tell the debug view how to get a pointer to memory for a custom type and display the button. Provide a way for a type to tell the debug view about a memory pointer it can use to open the binary memory view through the 'view binary data' button.

Applicable Scenarios
Better native pointer debug view support / customization
Describe alternatives you've considered
Trying to hack DebuggerTypeProxy & DebuggerDisplay attributes with different combinations
Additional context

The two main options I have are to use 'unsafe' pointers directly or to wrap a single field into a struct like you are suggesting. I have investigated both thoroughly and each has a problem:

For the unsafe pointer, I get the dereference + raw view for the opaque pointer which is not desirable.
For the struct wrapper, the issue is that while I can DebuggerDisplay the pointer value in hex to make it look like a pointer, I cannot get the 'view binary data' button to appear in the view for the variable. Yes, I can just expose a pointer property on the struct wrapper that when expanded shows the 'view binary data' button; however, its a hack. I do not understand how to get the button to appear for my custom struct variable even though vscode displays it for just about every other type of built in primitive.

After considering the options, the unsafe pointer is what I want instead of the struct wrapper, because the goal is not to hide the pointer / unsafe code from the user, but rather just provide some useful automatic utilities like a custom debug view for the object. Additionally, the struct wrapper is more code and confusion especially if we have more types of opaque like pointers.


build(esbuild): add build stage

Globbing pattern needs rework, as it excludes things like src/migrate.js due to the te part

Esbuild is overkill at this point for a small API.


Will DbConnection use a interface?

I've started building out a little ORM project for .Net Core and noticed that the DbConnection did not have any interfaces that I can wrap my providers behind. Is this something in the roadmap? If not, what is the expectation for unit testing, fakes by inheritance and what was the reasoning behind making this change from how .Net worked?
I tried looking for some documentation or issues related to this to help understand the direction you are going with DbConnection; came up empty.

Asked this in the wrong repository.


Add option to disable facter caching

The genesis framework currently caches the results of running facter. This becomes a bit of a problem if one of the facts change during a run. For instance, if we're running a target that updates the asset tag set in the SMBIOS this will not reflect in subsequent tasks since it has been cached.
This adds a configuration parameter to disable this cache and do a full facter lookup each time. This will obviously have a bit of a performance hit, but it's fairly small (at least in our genesis/facter setup). It defaults to false though, so existing configs should work the same way they did.
@Primer42 @roymarantz @qx-xp @gtorre

Is there anything that expects the facter class variable to exist? If this isn't an issue then 👍

It's only used internally in Genesis::Framework::Utils, apart from me trying some shady stuff in our internal tasks :) But I'll revert that before merging this.

LGTM

please add something to testenv/bootbox/puppet/modules/genesis/templates/config.yaml.erb or update some documentation explaining this config option


Linting.

Codecov Report

Merging #428 into master will decrease coverage by 1.01%.
The diff coverage is 45.09%.

@@            Coverage Diff             @@
##           master     #428      +/-   ##
==========================================
- Coverage   88.79%   87.78%   -1.02%
==========================================
  Files          22       23       +1
  Lines        1705     1727      +22
==========================================
+ Hits         1514     1516       +2
- Misses        191      211      +20

Flag
Coverage Δ

#unittests
87.78% <45.09%> (-1.02%)
:arrow_down:

Impacted Files
Coverage Δ

lineage/Analyze.py
10.57% <ø> (-0.75%)
:arrow_down:

lineage/Viterbi.py
100.00% <ø> (ø)

lineage/plotTree.py
0.00% <0.00%> (ø)

lineage/states/stateCommon.py
94.33% <ø> (-1.96%)
:arrow_down:

lineage/states/StateDistributionGaPhs.py
83.08% <66.66%> (-0.38%)
:arrow_down:

lineage/LineageInputOutput.py
87.75% <72.22%> (ø)

lineage/LineageTree.py
91.20% <85.71%> (-1.02%)
:arrow_down:

lineage/states/StateDistributionGaussian.py
75.55% <100.00%> (+2.22%)
:arrow_up:

lineage/tests/test_tHMM.py
100.00% <100.00%> (ø)

... and 2 more

Continue to review full report at Codecov.

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


document: create postgres user

createdb -d postgres
shoud be
createuser -d postgres

You are right, I will fix it later. Thank for your tip.


Add completion tests for records.

Tests to demonstrate the current functionality of completions for records.

Thanks. I can remove the draft-status. I had created this here to link it from the haskell/haskell-language-server#333

/azp run

/azp run

/azp run


WsFederation Token Encryption (Certificate)

Hi,
As you can read from WsFederation authentication doc:

Microsoft.AspNetCore.Authentication.WsFederation lacks support for token encryption, so don't configure a token encryption certificate...

We need this feature because of Signle-Sing-On auth system, that we want to implement in ours public web apps (Azure App Services working as relying party-trust in our scenario) and other internal (web based) products.
Do you have any plan to add this feature?

This issue was moved to AzureAD/azure-activedirectory-identitymodel-extensions-for-dotnet#966

@Tratcher, we need to use it as an external login provider and without ASP.NET Core Identity.
For this reason, i don't know if Azure AD Identity model ext is  the right repo.
Everything is working good following the doc, but we need to use the token encryption certificate to improve security on relying party-trust side (external web apps on azure app services).

They implement the low level protocol for us, encryption would be dealt with at that layer.


Use sidecar pool provider

What is the purpose of the change

See if sidecar can return pools missing from imperator pools query

@sunnya97 do you think we should actually use this? I think it's slightly less performant (+1-2s without any cache) than the imperator pools query since we have to calculate TVL ourselves from using our prices and the balances from the pools


fix freeze on error: stylusFile + ' already exists. Use -f to override.'

add aborting loop on another error.

the loop shouldn't be stopped. we want to continue processing existing files.

sorry i've misunderstood the issue. this actually fixes a bug.
thanks! :)


Ability to set default sorting column on List screen

I am looking for a method to set default sorting column in CRUD.
I need some option or method inside Resource class to define default sort column.
This is how it works in "classic" list screen:
https://orchid.software/en/docs/quickstart-sort-filter-table/
/**
 * Query data.
 *
 * @return array
 */
public function query(): array
{
    return [
        'posts' => Post::filters()->defaultSort('id')->paginate()
    ];
}

How would you like it to look? I'm not sure if we need a default property, but we can add one.
Now this can be done using a filter:
/**
 * @param Builder $builder
 *
 * @return Builder
 */
public function run(Builder $builder): Builder
{
    return $builder->defaultSort('id')
}

We could add default filters, like the trash filter.
For example:
$builder->defaultSort($builder->getModel()->getKeyName(), 'asc');
// or
$builder->defaultSort($builder->getModel()->getKeyName(), 'desc');

This will automatically match the primary keys and will not be visually displayed in any way.
And adding to the resource will look like this:
/**
 * Get the filters available for the resource.
 *
 * @return array
 */
public function filters(): array
{
    return [
        RoleFilter::class,
        DefaultSortDesc::class
    ];
}

What do you think about this?

I thought about following options:

$defaultSortColmn and $defaultSortOrder properties on Resouce class
public function defaultSortColumn(): string and defaultSortOrder() on Resource class
or even some method on Resource class to manipulate list query Builder $builder - maximum flexibility

For my current situation option 1 (new props) would be enough.
Regarding your proposal about adding filter witch manipulates with $builder - it sounds like "option 3"!
The only doubt that I fill "Filter" semantically is something that filters out records, but not sorting them, not changing the order.

Unfortunately, these modifiers are already named as Filters.
Using them will give you a lot more flexibility than simple properties.
I tried my own version and it may not always fit, since I want to indicate the column, maybe?
    public function filters(): array
    {
        return [
            new DefaultSort('asc', 'id'),
            new DefaultSort('desc'), // auto detect key
        ];
    }

I've just used in my project - works as expected! Thanks!


[Bug] on_schema_change failure with alternate schema

Is this a new bug in dbt-athena?

[X] I believe this is a new bug in dbt-athena
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
I have an incremental table int_nightly_sales, that is an intermediate table in our workflow.
When attempting to add a column to this table, I get a failure:
16:17:13  dbt.adapters.athena.constants adapter: Error running SQL: alter table `base_db_intermediate`.`int_nightly_sales`
          replace columns (<columns>)

16:17:13  On model.daily_sales_sync.int_nightly_sales: Close
16:17:13  Runtime Error in model int_nightly_sales (models/intermediate/upload/int_nightly_sales.sql)
  FAILED: SemanticException [Error 10072]: Database does not exist: base_db

The table should actually exist on: base_db_intermediate as called out in the config:
{{ config(
    materialized='incremental',
    incremental_strategy='append',
    partitioned_by=['processed_on'],
    on_schema_change='sync_all_columns',
    schema='intermediate'
) }}

I dug through the adapter source and could not find a culprit on this, but the odd thing is the first message references the correct db, where the semantic error thrown references the base_db (which does not actually contain anything at all).
Expected Behavior
The table exists on the proper database in athena, the adapter should reference the correct database instead of the base database for on_schema_change.
Steps To Reproduce

Create an athena dbt project with a base schema (base)
Create an incremental model with a different schema (sub) -> base_sub
dbt build
Add a column to the model
dbt build

This should fail as described.
Environment
- OS: Mint
- Python: 3.10.12
- dbt: 1.8.4
- dbt-athena-community: 1.8.4

Additional Context
No response

Hello @magno32 ! Apologies on the delayed response - we didn't realize that our github triage bot wasn't working correctly so I wasn't see new tickets on this repo. That's fixed now.
I'm struggling to recreate this error so would like to ask some follow up questions:

did you create the incremental model first in base_db and then moved it to base_db_intermediate?
What is the schema that the table int_nightly_sales is trying to select from? (is it selecting tables in base_db?)


How do I ignore a Terrascan rule?

Suppose I am running MSDO using a task in my ADO build pipeline.  Suppose Terrascan reports an issue?  Suppose that we accept the risk of this issue and no longer want MSDO to report it?  How can I configure this ADO task to ignore the Terrascan rule?  Terrascan has ways to allow this (see: https://runterrascan.io/docs/policies/policies/), but I don't see how to do this with MSDO.  TIA.

Use environment variables to configure Terrascan:
    - task: MicrosoftSecurityDevOps@1
      displayName: 'Run Microsoft Security DevOps'
      env:
        TERRASCAN_SKIPRULES: 'AC_DOCKER_0041'
      inputs:
        break: true

This is explained in the FAQ here: https://github.com/microsoft/security-devops-action


Avoid two candidates with same value.

Types of changes

[ ] Docs change / refactoring / dependency upgrade
[ ] 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)

Motivation and Context / Related issue

How Has This Been Tested (if it applies)

Checklist

[x] The documentation is up-to-date with the changes I made.
[x] I have read the CONTRIBUTING document and completed the CLA (see CLA).
[x] All tests passed, and additional code has been covered with new tests.

Supposed to fix https://github.com/facebookresearch/nevergrad/issues/976

Supposed to fix https://github.com/facebookresearch/nevergrad/issues/976


Add Priyanshu to Before I die

I'm adding Priyanshu to Before I Die with images and text.

@ignoreintuition, @itzmidinesh, @wagenrace, @elshaek
Anyone Kindky review it.


Failed to start API Mock endpoint TypeError: Cannot read property 'includes' of undefined on cli version 7.6.20

Before opening, please confirm:

[X] I have installed the latest version of the Amplify CLI (see above), and confirmed that the issue still persists.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
v12.22.10
Amplify CLI Version
7.6.20
What operating system are you using?
Mac
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
none
Amplify Categories
function, api
Amplify Commands
Not applicable
Describe the bug
Running
amplify mock api --timeout 30
Cause the following error
Failed to start API Mock endpoint TypeError: Cannot read property 'includes' of undefined
while rolling back to cli version 7.6.19 does not cause the issue.
Expected behavior
Mock should work
Reproduction steps

run amplify mock

GraphQL schema(s)

# Put schemas below this line

Log output

# Put your logs below this line

Additional information
No response

Hello, any updates on this?

I'm still having troubles with this too, I need to revert my amplify cli version to 7.6.19 every time I want to run the mock API 😞

same issue here :(

Is this maybe resolved in version 10.0.0 released a couple of days ago?

This is still very much a problem even on the newest version of cli 10.4.0. I'll send the reports via email that I just got from the newest cli.

This suddenly started hapenning now in 10.5.0 as well as 10.0.0, node version v14.19.1 or even with v19.2.0
Is there a timeline to fix this, please?

If I downgrade, my entire project is broken. Is there a workaround? Is there someplace in my project that I can put an empty array that will fix this? I'm currently blocked.

If I downgrade, my entire project is broken. Is there a workaround? Is there someplace in my project that I can put an empty array that will fix this? I'm currently blocked.

@mkane91301 we use the following workaround in our project:
we nave two nvm environments and we switch between them. So we use the newer one (v8.4.0) to do the changes to our amplify environments (e.g. amplify push) and the old one (7.6.19 - the latest version that doesn't have this bug) to start the amplify mock. After amplify mock is started it overrides the API.ts, mutations, queries, etc and I believe that is what breaks your project. But we simply discard those changes and after that we start our app that points to the mock GraphQL server that was started by an older amplify. With that discard we essentially have a project generated by the new version and the mock server run by the old amplify version.
Of course it is just a workaround and we would really appreciate if the bug would be fixed and we could get rid of this complicated test run setup ❤
Some details about how to setup two separate amplify environments and quickly switch between them.
Prerequisite: make sure you don't have a global node installation, otherwise amplify global installation will end up there instead of the selected environment. Install nvm
nvm install 14.18.3
npm install -g<EMAIL_ADDRESS>nvm install 14.19.3
npm install -g<EMAIL_ADDRESS>
Now you can quickly switch:
nvm use 14.18.3
# Now using node v14.18.3 (npm v6.14.15)
amplify --version
# 7.6.26
nvm use 14
# Now using node v14.19.3 (npm v6.14.17)
amplify --version
# 8.4.0

This is even more bizarre that I have two projects with the same versions of all major libraries, and it works for one, not for the other. The package.json files are identical.

Still a problem in the latest version. Just downgraded back to 7.6.19...

@josefaidt Is there a timeline to resolve this, please? In two weeks, this issue will have a birthday. 🎂

Facing the same issue here :///
Seems like people at AWS are more worried about their jobs then their customer's issues.

@josefaidt same question, is there a timeline for when this issue will be fixed? Thanks!
We still have to use version 7.6.19 when we want to run tests and create a lot of workarounds for this.


html-metadata returns all fields as undefined for specific url

Hi,
I have been using html-metadata and thanks for such a wonderful software. I noticed that when used on the following url https://www.cnet.com/special-reports/vr101/ - it gives all fields as undefined.
Can you please look into the issue.
My code is
var scrape = require('html-metadata');
var url = process.argv[2];
scrape(url).then(function(metadata){
console.log("************************");
console.log(metadata);
});
and the output I get for this program is
parse() is deprecated, use toJson()

{ openGraph:
{ site_name: undefined,
title: undefined,
description: undefined,
url: undefined,
image:
{ url: undefined,
type: 'image/jpeg',
width: '630',
height: '315' },
app_id: undefined,
type: 'article' },
twitter:
{ card: 'summary_large_image',
creator: undefined,
site: undefined } }

Thanks for reporting!
The underlying issue with that url is that it looks like they're using a templating language to create their html but the values for some reason aren't getting replaced and it's putting the literal template tags in instead:
<!-- OpenGraph sharing tags -->
    <meta property="og:site_name"     ng-attr-content="{{share.siteName}}" />
    <meta property="og:title"         ng-attr-content="{{share.title}}" />
    <meta property="og:description"   ng-attr-content="{{share.description}}" />

But obviously we shouldn't be returning undefined! :)

Yes. So what would be returning and what time it may take to fix this?.
Regards,
S.Satyanath
On 06-02-2017 15:29, M. Volz wrote:

Thanks for reporting!
The underlying issue with that url is that it looks like they're using
a templating language to create their html but the values for some
reason aren't getting replaced and it's putting the literal template
tags in instead:
|    |
But obviously we shouldn't be returning undefined! :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/wikimedia/html-metadata/issues/54#issuecomment-277635795,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFQiA6SEM30g3D4ng_9rFWrrOdaJQmCuks5rZu7ogaJpZM4LzrMv.

Hi,
I will fix it returning undefined, and instead you will get cleaner, but not very rich metadata. It looks like they have a programming error on their end :/ I'll let you know when the update is published.

Thanks a lot for the quick response.
Regards,
S.Satyanath
On 06-02-2017 15:43, M. Volz wrote:

Hi,
I will fix it returning undefined, and instead you will get cleaner,
but not very rich metadata. It looks like they have a programming
error on their end :/ I'll let you know when the update is published.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/wikimedia/html-metadata/issues/54#issuecomment-277639168,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFQiA6KsWBW2zv8fSA5J2cyk4Q5fmarbks5rZvI4gaJpZM4LzrMv.

I think this has actually been resolved for a while, so closing :).


az vm secret format returns nothing

Describe the bug
az vm secret format returns nothing. With -o json, it doesn't seem to return anything that's usable by VMs.
To Reproduce
Create a secret in a vault:
$ az keyvault secret set --vault-name examplevault --name testfoo --value this-is-a-value
Value
---------------
this-is-a-value
$ az keyvault secret list-versions --vault-name examplevault -n testfoo -o json
[
  {
    "attributes": {
      "created": "2018-09-10T13:21:15+00:00",
      "enabled": true,
      "expires": null,
      "notBefore": null,
      "recoveryLevel": "Purgeable",
      "updated": "2018-09-10T13:21:15+00:00"
    },
    "contentType": null,
    "id": "https://examplevault.vault.azure.net/secrets/testfoo/deadbeefcafedeadbeefcafedeadbeef",
    "managed": null,
    "tags": {
      "file-encoding": "utf-8"
    }
  }
]

Attempt to format it for VM use:
$ az vm secret format --secret https://examplevault.vault.azure.net/secrets/testfoo/deadbeefcafedeadbeefcafedeadbeef

$ az vm secret format --secret https://examplevault.vault.azure.net/secrets/testfoo/deadbeefcafedeadbeefcafedeadbeef -o json
[
  {
    "sourceVault": {
      "id": "/subscriptions/00000000-1111-2222-3333-444444444444/resourceGroups/resourcegroupfoo/providers/Microsoft.KeyVault/vaults/examplevault"
    },
    "vaultCertificates": [
      {
        "certificateUrl": "https://examplevault.vault.azure.net/secrets/testfoo/deadbeefcafedeadbeefcafedeadbeef"
      }
    ]
  }
]
$

Expected behavior
az vm secret format, as the usage is described in e.g. https://github.com/MicrosoftDocs/azure-docs/blob/master/articles/virtual-machines/linux/tutorial-automate-vm-deployment.md , should return some value that can be added to az vm create --secrets.
Environment summary
macOS 10.13.6, azure-cli (2.0.45) installed via Brew, on Python 3.7.

Apparently the output generated by -o json can be piped into a file, then used with --secrets @..., i.e.
$ az vm secret format --secret https://examplevault.vault.azure.net/secrets/testfoo/deadbeefcafedeadbeefcafedeadbeef -o json > secret-payload.json
$ az vm create --other-parameters-here --secrets @secret-payload.json

However it's important to note (as I found out the hard way) that secrets other than certificates are not supported at present by the VM creation process.
That is, you can't use this method to inject an arbitrary secret value into your VM.
The bug still stands though; az vm secret format should output something usable without -o json.

Hello @akx
What is your default output format? The default CLI output format is json. I imagine your default output format is table.
Unfortunately, output formats other than json/jsonc can lose information because there isn't always a clear way to format certain types of data. One easy problem is how to clearly output nested json information in a tsv or table format.
Here is my output when I run the same command.
$ az vm secret format --secret $my_secret --keyvault tosinkeyvault
[
  {
    "sourceVault": {
      "id": "/subscriptions/00977cdb-163f-435f-9c32-39ec8ae61f4d/resourceGroups/ovaTestGroup/providers/Microsoft.KeyVault/vaults/tosinkeyvault"
    },
    "vaultCertificates": [
      {
        "certificateUrl": "https://tosinkeyvault.vault.azure.net/secrets/password/e5724bf40e7b432b8a793e3f6ead89f3"
      }
    ]
  }
]

$ az vm secret format --secret $my_secret --keyvault tosinkeyvault -o table

$ az vm secret format --secret $my_secret --keyvault tosinkeyvault -o tsv
	1

As you can see, no information is outputted with -o table. If your default output isn't -o table or -o tsv, let me know, otherwise this is the expected behavior.
Concerning using arbitrary secrets during VM creation, this is a service side issue.
I will be closing this issue, but please let me know if you have any more questions.

It would probably be user friendly to document that this command always requires JSON output, or maybe even force it to always output as JSON, if all other formats are utterly useless? :)

Hello @akx. Thanks for the feedback. However, this issue of certain output formats omitting nested data applies to other commands. You might find our docs on output formats helpful, they mention how certain output formats omit nested data.
The query option can be very helpful for extracting desired fields from / transforming data in the result.
Here is a helpful doc that explains how --query works.
For example, so to get at the sourceVault id in the command above:
ID=$(az vm secret format --secret $my_secret --keyvault tosinkeyvault --query "[].sourceVault.id" -o tsv)

This will give you the sourceVaultID without the quotes.

I'm aware of various output formats being available, thank you :)
However, I would say az vm secret format is a special case, as it is a command primarily (or rather, solely) meant for retrieving the "blob" of data that is to be used with az vm create, as I discussed in this comment a month ago, and the data is absolutely not usable for that command in any other format than json.

@akx, no worries. I was just referring you to the doc because in the description of the tsv and output formats, it is mentioned that these formats omit nested data.
I created an issue to warn users when -o table and -o tsv formats are used.


commit lerna script results

@sidoruk-sv why deps downgrades all over?

@hugebdu I'm just committing the result of post-push hook for this repo (https://github.com/wix/lerna-script/blob/master/package.json#L28)
I think it is the result of one of syncModules (https://github.com/wix/lerna-script/tree/master/tasks/modules) or dependencies.sync (https://github.com/wix/lerna-script/tree/master/tasks/dependencies#syncpackageslog-promise)
that are defined in root lerna.js

indeed. managed deps those are.
have to be updated.

@hugebdu updated, also got errors when tried to run lerna-script depcheck so removed unused lerna from packages.


10/21 Manual Review - Communication

9.1 Link created by Dr. Sunshine Menezes is broken
9.4 Twitter; there’s been some discussion about not using twitter in the lab
Some more discussion as to whether or not students can feel free to post on twitter if they would like, or if it should best be avoided?
9.6 Unfinished

merge 9.4-9.5 into general social media section
posit resources to include in here as well


[CSS] class was overwritten by tailwindcss3 Preflight

TuSimple/naive-ui version (版本)
2.24.6
Vue version (Vue 版本)
3.2.29
Browser and its version (浏览器及其版本)
Chrome Version 97.0.4692.99 (Official Build) (x86_64)
System and its version (系统及其版本)
macOS 12.1 (21C52)
Node version (Node 版本)
v16
Reappearance link (重现链接)
https://github.com/uruz-7/tailwindcss3-naive-ui-reproduction
Reappearance steps (重现步骤)
pnpm install
pnpm vite

Expected results (期望的结果)
background color is not overwritten
Actual results (实际的结果)
background color is overwritten
Remarks (补充说明)
This issue (#1808) still occurs with latest version "naive-ui": "2.24.6", "tailwindcss": "3.0.16".
这个issue(#1808)在最新版本里仍可重现。
Tailwind css preflight: https://tailwindcss.com/docs/preflight

可以尝试一下这个方法
https://www.naiveui.com/zh-CN/os-theme/docs/style-position

It worked. Thanks!

you can use the following solution. 可以尝试一下这个解决方案。 https://www.naiveui.com/en-US/os-theme/docs/style-position https://www.naiveui.com/zh-CN/os-theme/docs/style-position

链接失效了

you can use the following solution. 可以尝试一下这个解决方案。 https://www.naiveui.com/en-US/os-theme/docs/style-position https://www.naiveui.com/zh-CN/os-theme/docs/style-position

链接失效了

updated

you can use the following solution. 可以尝试一下这个解决方案。 https://www.naiveui.com/en-US/os-theme/docs/style-position https://www.naiveui.com/zh-CN/os-theme/docs/style-position

链接失效了

updated

打开是官网首页

https://www.naiveui.com/zh-CN/os-theme/docs/style-conflict

<EMAIL_ADDRESS><EMAIL_ADDRESS>

The real reason is that tailwind has a preset default style of Preflight, which is friendly to new projects, but when we integrate tailwindcss into existing projects, it will cause style conflicts, we only need to disable this tailwindcss Preflight The style settings in your tailwind.config.js or tailwind.config.cjs
module.exports = {
  corePlugins: {
    preflight: false,
  }
}

Refer to tailwind css official website for explanation：
https://tailwindcss.com/docs/preflight#disabling-preflight


NPM Module update + Storage Display

I added a storage Usage display option!
PS. If you decide to use this a credit in the readme would be nice


ETS2_v<IP_ADDRESS>

Which game?
ETS2
What is your game version?
V<IP_ADDRESS>
Are you using any map mods?
No
If yes to the above, please list what mod and their version.
No response
Link to Map Tiles
https://drive.google.com/file/d/1ovjSUl_pW36FtkaZpIMri0nNUebqOHIm/view?usp=drive_link
Have the tiles been tested?

[X] I have tested and read the Contributing document.

Additional Information
No response

这是来自QQ邮箱的假期自动回复邮件。你好，我最近正在休假中，无法亲自回复你的邮件。我将在假期结束后，尽快给你回复。

Hey there, just now recalled this from my email.
As a matter of fact, I made a branch, 149 and updated it just about a week ago here: https://github.com/Unicor-p/SCS_Map_Tiles/commit/55068018be7874f571d825e0a9a122d9d7ea4e91 - These were specifically for ETS2 1.49, however I'll go ahead and leave this issue open in the meantime in case my export may have issues. (I hadn't looked over them personally on my end, but I did take a quick peak at what you uploaded.)
I'm still waiting for upstream on ts-map for the generation to be fixed for ATS. However, I appear to be the only one who has all of the DLC for ATS to test, so it seems to be slow-going on that front. On top of a looming DLC that may be released in a few short months for ATS even.


Consider simplified docs for operations and leverage routes for path arguments

Looking at some concrete examples of some rest layer cadl I find things like this quite often:
  @summary("Check if there are any client connections connected for the given user.")
  @doc("Check if there are any client connections connected for the given user.")
  @route("/api/hubs/{hub}/users/{userId}")
  @head
  UserExists(
    @doc("""
Target hub name, which should start with alphabetic characters and only contain
alpha-numeric characters or underscore.
""")
    @path
    hub: string,

    @doc("Target user Id.")
    @path
    userId: string,
  ): ErrorDetail | void;

Observations:

I don't understand why we need summary and doc with the same string.
The docs directly on the parameters significantly reduce readability.
the @path on the params is redundant with the specification in the route.

I'd much rather be able to write something like this:
@doc("""Check if there are any client connections connected for the given user.
        {hub} Target hub name, which should start with alphabetic characters and only contain
            alpha-numeric characters or underscore.
        {userId} Target user Id. )
@route("/api/hubs/{hub}/users/{userId}")
@head UserExists(hub: string, userId: string): ErrorDetail | void;

I suspect this would just mean:

Adding support in the doc decorator to apply parts of it to arguments
It would require the @route decorate be able to add the @path to the args.
We just have to figure out what the use case is for @summary and @doc together.

Another interesting observation here is the fact that the text of one of the parameters is there to spell our the constraint on the string. I wonder if we could generate that text in some way from format decorator on the param? "which should start with alphabetic characters and only contain
alpha-numeric characters or underscore."

Types of documentation string:

Service Enduser (final API documentation)
Cadl spec writer (building bloc such as library base class/operations,  TrackResource, ResourceOperation usages)

Questions:  @doc vs.  // comment bloc
String templates (see other issue on this topic) to enable docs on sub-elements in parent elements
Emit summary from doc if explicit summary isn't provided?
Work item:
.

Infer @path from @route token so that @path is optional.
general string template support

#1973

Proposal a complete multiline format with property/parameters template reference.

template token on a new line with no proceeding

For models, template token refers to properties

@doc("""
  A person

  {name}: The person's name
  {age}: The person's age
""")
model Person {
  name: string;
  age: uint8;
}

for operations, template token refers to parameters

 @doc("""
  Rename a person.

  {from}: the current name of the person
  {to}: the new name for the person
  {self::returnType}: returns the person that was renamed.
""")
op rename(from: string, to: string): Person;

for templated types, token refers to Template parameters

@doc("""
  A resource model

  {TResourceProperties}: The resource property template type that goes with the resource
""")
  model ResourceModel<TResourceProperties>

Specialized Tokens:

Operation returns  `{self::return}"
Any others?

Open questions:

The current proposal only addresses the token to field reference issue. Do we need similar to JSDoc @xxx (maybe a subset) to add more classifications. This may remove the need to use specialized tokens as well as separate the template arguments doc from regular property doc.

@allenjzhang @nguerrera Please add your latest writeups

Also linking @bterlson's string template sketch that lines up with @allenjzhang's notes above: https://gist.github.com/bterlson/a23d1bd5f8f8f5afec3eb5bf937b93b6

Let's start with the string interpolation without JSDoc style token. The proposal and sample are as writeup above.
As for the API doc and dev doc from library, Nick has a good proposal with extern dec that can be unified with multi-line @doc proposal above.

I didn't have time to get into this when we spoke yesterday, but it is not possible to unify everything with @doc documentation. This is covered in the linked developer doc proposal: there are entities (building blocks) that exist only at Cadl-authoring time and these entities cannot have decorators, nor should they. If we tried to use @doc on an extern dec, that would mean that we were decorating decorators! This would greatly complicate things. I don't think it makes sense to reify these decorators in the modeling of entities that can be decorated just so that we can document them.
So, something other than decorators is needed to document decorators (and aliases, functions, etc.).  But then the problem is that people see the (subjectively) nicer syntax for those cases and invariably question why they can't use it to document API. And I sympathize with that.
It's incredibly difficult to explain the distinction between these two scenarios. They are technically very real and very different, but we often find ourselves selling Cadl to an audience that is only interested in reading inline API documentation that looks pretty and familiar. And it increasingly feels to me like I'm losing that sale when I get into explaining all this. So I'm thinking about ways to blur this distinction to the extent possible without compromising functionality.
I believe the only way to unify the syntax is in the other direction: towards doc comments for everything: API end-user and Cadl building block consumers. My basic idea is that some doc comments (or perhaps some portions of doc comment) will still compile down to decorator application so that they can still benefit from sidecars, decorator composition, and so on. I will try to write this up in a bit more detail.

I wrote up the idea I had here:
https://gist.github.com/nguerrera/a48a3d8f005eb5322683fc5ddc48c363

Commented on Nick's gist. Post here for visibility as well.
Having distinct parts come out looks a lot busier. I do agree that the comment block style looks a lot more familiar in code, but one of the goals is to be able to easily integrate out for doc which will be harder to do. We have already established the need for @doc for Cadl spec documentation purposes. I haven't seen good scenarios/reasons that @doc can't be used.

Having distinct parts come out looks a lot busier.

Can you elaborate on this? I don't know what this means.

I haven't seen good scenarios/reasons that @doc can't be used.

You mean apart from the developer documentation scenarios, right? You're saying we should continue with POR of doc comment for things like decorators and @doc for API docs?

Remaining work item:
https://github.com/microsoft/cadl/issues/1399

Reopening as this is what tracks the linked proposal on unification.

But also noting that the prereq developer docs proposal (Design: #131, Implementation: #719, PR: #1313) is done. 😁

I am not sure if this should be closed, do we have implementation issue(s) for the unification yet? cc @timotheeguerin.

I think maybe we should close #1399 and replace it with what we discussed before the holidays.

I didn't create an issue, I can do that now


Fixing test by updating snapshot for TouchableHighlight

Tests were failing due to not updated snapshot about TouchableHighlight.
Test Plan
Run npm test.

Thanks for the PR, I had a look at why the snapshot changed and it is caused by https://github.com/matejstrasek/react-native/commit/e69169982021261e138add21506dea11dd91b7e4 so this change is fine.
@javache Could you land this? Also does this test not run on fb infra? I'm surprised it got broken.

@facebook-github-bot shipit


Additional modules required beyond pip3 install -i https://test.pypi.org/simple/ bd-export-spdx2.2

bd-export-spdx2.2 requires the aiohttp and lxml modules and they are not installed with bd-export-spdx2.2 using test.pypi.org/simple.
However, when I loaded those modules and attempted to run bd-export-spdx2.2, I got:
2021-10-31 16:17:50,334:INFO:Using a session with a 15 second timeout and up to 3 retries per request
BLACK DUCK SPDX EXPORT SCRIPT VERSION 0.21
2021-10-31 16:17:50,336:WARNING:ssl verification disabled, connection insecure. do NOT use verify=False in production!
2021-10-31 16:17:50,416:INFO:success: auth granted until 2021-10-31 18:17:50.413098-04:00
Working on project '' version ''
Getting component list ... (67)
Traceback (most recent call last):
File "/usr/local/bin/bd_export_spdx", line 11, in
sys.exit(run())
File "/usr/local/lib/python3.6/site-packages/export_spdx/main.py", line 133, in run
process.process_project(project, version, toppackage, hierarchical_bom, bearer_token)
File "/usr/local/lib/python3.6/site-packages/export_spdx/process.py", line 239, in process_project
comp_data_dict = asyncio.run(async_main(bom_compsdict, bearer_token, version))
AttributeError: module 'asyncio' has no attribute 'run'
FYI.

OK.   Apparently, there is a minimum requirement for Python here.
With Pete Jalajas (Synopsys) helped me with this.   The following should get people up and running if there is an issue:
In this example, Andy made a /mnt/hub/spdx and /mnt/hub/spdx/pipenv directories.  Scripts to run SPDX 2.2 were located in /mnt/hub/spdx and the pipfile was created and located in /mnt/hub/spdx/pipenv.   This latter item could be anywhere.  It did not need to be a subdirectory.  It could be in ~/piptahdah or whatever.
Server (Red Hat):
sudo yum install rh-python38.x86_64 rh-python38-python-devel.x86_64 rh-python38-python-pip.noarch
First Time for user as user:
cd /mnt/hub/spdx/pipenv
pip install --user pipenv
~/.local/bin/pipenv --python /bin/python3.8 shell
pip3 install -i https://test.pypi.org/simple/ --extra-index-url https://pypi.org/simple bd-export-spdx2.2
<Done. Yeah!>
cd /mnt/hub/scripts/spdx and Run scripts
Exit
to go back in:
cd /mnt/hub/scripts/spdx/pipenv   (Where your Pipfile is located)
~/.local/bin/pipenv shell


the song bug

Describe your bug here. If you are modding, try testing your bug in a clean version of the engine instead. Also, if you're using an older version of JSE, please try the latest version/action build. Also, be sure to check the pinned 'JS Engine: Known Issues' issue, to check if your issue hasn't already been found!
simply entering a song crashes the game
with no crash message (0.32.2)
Command Prompt/Terminal/Crash logs (if existing)
none

Have you identified any steps to reproduce the bug? If so, please describe them below in as much detail as possible. Use images if possible.
No response
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
none
If you use 1.12.0 or earlier, did you have Optimized Chart Loading turned on?
None
Did you check for any similar issues to what you're reporting? Check, then come back here. If there is a similar issue, then do not report the issue, otherwise it will be marked as a duplicate.
Yes

duplicate of #509


Small doc changes

Add a little bit more to the README
Rename .catalog.yaml to catalog.yaml as dot-files can't be attached to releases.
Update design.md

/lgtm
Let's keep the comments for later conversations, please.


Year view

Implement a year view into the gallery view #4.

Full calendar year

Months and days

Indication whether a month/day has photos

Heat map
Link to open in month/individual view

Implemented as part of #12.


[cider] Flattening the interpreter

This is a tracking issue for the work being done toward #1183.

[x] Translate the normal IR into the internal flat representation (FIR, perhaps pronounced "fear")

[x] Print out the groups and assignments as a sanity check (#1547, #1572)

[x] Instantiate the program environment (#1578)
[x] Print the program state in a nice human readable way (#1580)
[ ] Program counter

[ ] Sketch a basic design for the program counter in the single component case. Must account for the control flow stack and the state machine for control flow operators (e.g. seq)

[ ] Primitives

[ ] Flesh out the new interface

[ ] Instantiating a primitive requires giving it either a list of ports or an offset into the global list. May also wish to add a sanity check using the names of ports since otherwise changes in ordering within primitive definitions could break things. Probably just want to make a good macro for this.

[ ] Translate primitives implementations to the new interface

Looks great. One minor thing we've talked about here is the thought that we could run a primitive-free program after the "PC" step. It is of course really hard to write a primitive-free Calyx program that does anything, but maybe it works to just write the input directly into the output. But if we could write and run such a program, it would go some way to indicating that we are on the right track before trying to tack on primitives.
One possible high-level design for the PC thing is: we do not even try to represent the "intra-group" part of the state. That is, if the conceptual interpretation loop is:
for stmt in control:  # control loop
    for cycle in cycles:  # cycle loop
        while not changed:  # stabilization loop
            propagate combinational assignments

…then the "PC" would only represent progress through the outermost (control) loop. We would not attempt to have a representation for being "part way through the evaluation of a given group" for example. The result is that the PC could be pretty simple, i.e., just $n$ pointers to leaf statements in the control statement for each cell in the program (where $n$ maybe zero for inactive cells). Would this make sense?

That makes sense! Too bad it's not easier to write a primitive-free program. Maybe we have to forge ahead and get registers or something working before we have a v0 that can execute anything…
I'm not quite sure I understand the "stack" thing. Maybe I'm thinking about this wrong, but if (for example) multiple threads activate the same group, the "PC" would point to those many different group activations (i.e., control statements). So this would be enough information to always know what happens next, at the granularity of the control loop. Does this make sense?

@EclecticGriffin is this tracker still being actively used? If not, we should close this and migrate to the new one.

Oops yeah forgot about this one, we can close it. All the current stuff is happening in #1913


Installation failed

The context:

Virtualbox 6.1.40_Debian r154048
Running a brand new Windows 11 obtained at https://developer.microsoft.com/en-us/windows/downloads/virtual-machines/
This installer: https://github.com/diskuv/dkml-installer-ocaml/releases/download/v1.1.0_r2/setup-diskuv-ocaml-windows_x86_64-1.1.0.exe

The log: http://komar.in/files/setup-userprofile.full.log
The problem is about this I think:
checking whether fma works... no
configure: error: fma does not work, enable emulation with --enable-imprecise-c99-float-ops
FATAL: ./configure failed

Also, the "red text" not included in log:
Setup did not complete because an error occurred.
MSYS2 command failed! Exited with 107. Command was: env MSYSTEM=CLANG64 MSYSTEM_PREFIX=/clang64 HOME=/home/User PATH=/clang64/bin:/usr/bin:/bin env PATH=/c/Program Files/Git/cmd:/clang64/bin:/usr/bin:/bin DKML_TMP_PARENTDIR=/c/Users/User/AppData/Local/Temp/PAFADE~1/0 env DiskuvOCamlVarsVersion=2 DiskuvOCamlHome='/c/Users/User/AppData/Local/Programs/DISKUV~1' DiskuvOCamlBinaryPaths='/c/Users/User/AppData/Local/Programs/DISKUV~1/usr/bin:/c/Users/User/AppData/Local/Programs/DISKUV~1/bin' DiskuvOCamlMSYS2Dir='/' DiskuvOCamlDeploymentId='v-1.1.0;ocaml-4.14.0;opam-2.2.0~alpha0~20221228;inotify-36d18f3dfe042b21d7136a1479f08f0d8e30e2f9;msys2-999CED2C8F5FF6A2;docker-B01818D2C9F9286A;bins-952D9673A6642899;stubs-4E6958B274EAB043;toplevels-80941AA1C64DA259' DiskuvOCamlVersion='1.1.0' WITHDKML_ENABLE=OFF OCAMLLIB= TOPDIR=/c/Users/User/AppData/Local/Temp/7Z75A1~1/sg/NETWOR~1/WINDOW~1/dkmldir/vendor/drc/all/emptytop sh /c/Users/User/AppData/Local/Temp/7z75A1B418/sg/network-ocamlcompiler/windows_x86_64/install-ocaml.sh /c/Users/User/AppData/Local/Temp/7Z75A1~1/sg/NETWOR~1/WINDOW~1/dkmldir 15553b77175270d987058b386d737ccb939e8d5a windows_x86_64 /c/Users/User/AppData/Local/Programs/DISKUV~1 --disable-imprecise-c99-float-ops

at Invoke-MSYS2Command, C:\Users\User\AppData\Local\Temp\7z75A1B418\sg\network-ocamlcompiler\windows_x86_64\dkmldir\vendor\drd\src\windows\UnixInvokers\UnixInvokers.psm1: line 165
at Invoke-MSYS2CommandWithProgress, C:\Users\User\AppData\Local\Temp\7z75A1B418\sg\network-ocamlcompiler\windows_x86_64\setup-userprofile.ps1: line 856
at <ScriptBlock>, C:\Users\User\AppData\Local\Temp\7z75A1B418\sg\network-ocamlcompiler\windows_x86_64\setup-userprofile.ps1: line 1117
at <ScriptBlock>, <No file>: line 1
at <ScriptBlock>, <No file>: line 1

[Advanced] MSYS2 commands can be run with: C:\Users\User\AppData\Local\Programs\DISKUV~1\tools\MSYS2\msys2_shell.cmd

Bug Reports can be filed at https://github.com/diskuv/dkml-installer-ocaml/issues
Please copy the error message and attach the log file available at
  C:\Users\User\AppData\Local\Programs\DISKUV~1\setup-userprofile.full.log

FATAL [a0d16230]. A transient failure occurred.

Root cause: The conformant command had exit code 1:
                                           C:\Users\User\AppData\Local\Temp\7z75A1B418\sg\network-ocamlcompiler\windows_x86_64\setup-userprofile.bat
                                             -AllowRunAsAdmin
                                             -InstallationPrefix
                                             C:\Users\User\AppData\Local\Programs\DISKUV~1
                                             -MSYS2Dir
                                             C:\Users\User\AppData\Local\Programs\DISKUV~1\tools\MSYS2
                                             -OpamExe
                                             C:\Users\User\AppData\Local\Programs\DISKUV~1\bin\OPAM-R~1.EXE
                                             -DkmlPath
                                             C:\Users\User\AppData\Local\Temp\7Z75A1~1\sg\NETWOR~1\WINDOW~1\dkmldir
                                             -GlobalCompileDir
                                             C:\Users\User\AppData\Local\Temp\7Z75A1~1\sg\STAGIN~1\generic
                                             -NoDeploymentSlot -DkmlHostAbi
                                             windows_x86_64 -TempParentPath
                                             C:\Users\User\AppData\Local\Temp\PAFADE~1
                                             -SkipProgress

>>> A transient failure occurred. <<<

dkml-package.bc: [ERROR] FATAL [5f927a8b].
                         A transient failure occurred.

                         Root cause: The command had exit code 20:
                                       C:\Users\User\AppData\Local\Temp\7z75A1B418\bin\dkml-install-user-runner.exe
                                         install-user-network-ocamlcompiler --verbosity=info
                                         --color=always --prefix
                                         C:\Users\User\AppData\Local\Programs\DiskuvOCaml
                                         --staging-files
                                         C:\Users\User\AppData\Local\Temp\7z75A1B418\sg

                         >>> A transient failure occurred. <<<

Press "y" and ENTER to exit the installer.

update: fixed the log link

See https://github.com/diskuv/dkml-installer-ocaml/issues/27 for VirtualBox
I’d also like you to try out a prerelease instead of that 1.1.0_r2 version … let me find the link …

The prerelease is available at:

v1.2.0-prerel7 Installer: unsigned-diskuv-ocaml-windows_x86_64-i-1.1.0.exe
v1.2.0-prerel7 Uninstaller: unsigned-diskuv-ocaml-windows_x86_64-u-1.1.0.exe

Well, yesterday you asked me not to use prerelease builds, but I'll try anyway. Thanks.

Hah. Yes I did. But now the releases have been revamped … you can only see prerelease builds that have undergone lots of testing that I’ve verified at least once. Before 99% were broken builds.

Installation succeed (with v1.2.0-prerel7 I think), utop works.
Just for history, I found a very confusing behaviour. The terminal with installer running suppressed the output if Alt+3 (without F) was pressed (which is a hotkey for my window manager). I restarted the installation a couple of times because of this before I figured it out. Have no idea how it works.

And just to be precise: here are two commands a VirtualBox user should run in PowerShell before the installation:
mkdir C:\ProgramData\DiskuvOCaml\conf\
Set-Content -Path "C:\ProgramData\DiskuvOCaml\conf\ocamlcompiler.sexp" -Value "((feature_flag_imprecise_c99_float_ops))"

Thanks for those instructions. I will make that prominent somewhere.
For the version, you should be able to run dkml version to see that you have v1.2.0-prerel7.

Yep, it's 1.2.0~prerel7


NPM, Browserify support

Also might consider NPM support for individuals looking for browserify support

NM, I see it's supported. But I would suggest including it in the README :)


[CH] Implement native orc reader without arrow dependency.

In CH, ORC output format use native orc writer without arrow dependency. So we can also implement native orc reader in ORC input format in similar way.  It should has better performance compared to current implementation.

After implementing native orc reader without arrow dependency, the cost of transform ORC ColumnVectorBatch to arrow Table would disappear, which will speedup orc reader by 1.7x

wip in https://github.com/bigo-sg/ClickHouse/tree/ch_gluten_2583

flamegraph using new native implementation of ORC input format


rm dependabot coverage

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 47.46%. Comparing base (39e8f92) to head (b48323a).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##           master     #309   +/-   ##
=======================================
  Coverage   47.46%   47.46%
=======================================
  Files         232      232
  Lines        3032     3032
  Branches     1493     1493
=======================================
  Hits         1439     1439
  Misses        411      411
  Partials     1182     1182

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


ajv-formats for draft-04.

Heya,
I've just been caught out by some format validation on my draft-04 spec.
format: date was introduced in draft-7 - but was not present in draft-4.
format: date-time appears to not have been introduced in draft-6 or draft-7 - but I can't figure out where to find it originally defined.
Reading through the AJV docs; due to the problems inherent with format, it's been moved to ajv-formats as a standalone library.
Appreciate this may be a noddy question, but how would I work out which of those formats were compatible with draft-04 (as date-time appears to be valid, but date and time individually are not)?
Happy to assist if the job is reading through a spec and workout how to leverage a subset of ajv-formats that is draft-04 specific.

To answer my own question:
The specification for the formats supported in draft-04 is present here:
https://datatracker.ietf.org/doc/html/draft-fge-json-schema-validation-00#section-7.2
To use these with draft-04:
`import Ajv from "ajv-draft-04"
import addFormats from "ajv-formats"
const ajv = new Ajv();
addFormats(ajv, ["date-time","email","hostname","ipv4","ipv6","uri"])`


fix lint and sec update

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fixing the omitempty tag

Fixes https://github.com/GoogleCloudPlatform/kubernetes/issues/7730 and https://github.com/GoogleCloudPlatform/kubernetes/issues/6841
Fixing the omitempty tag for v1beta3 and v1 APIs.

@bgrant0607

Thanks much. I reviewed the whole v1beta3 file. Please make equivalent changes to v1.

Updated both v1beta3 and v1 as per comments, modulo one question regarding ObjectMeta.

LGTM


pkg/path: godoc package comment appears stale and misleading

What version of CUE are you using (cue version)?
n/a
Does this issue reproduce with the latest stable release?
Yes: 0.7.0.
What did you do?
As a CUE user, I often rely on the docs at https://pkg.go.dev/cuelang.org/go/pkg in lieu of a more CUE-centric site.
I accessed the top-level list of packages (https://pkg.go.dev/cuelang.org/go/pkg#section-directories) and saw mention of a non-existent package, filepath:

path
Package filepath implements utility routines for manipulating filename paths in a way compatible with the target operating system-defined file paths.

I accessed the package overview on its dedicated page (https://pkg.go.dev/cuelang.org/go@v0.7.0/pkg/path#pkg-overview), and saw the same initial paragraph as above, followed by an additional paragraph which mentions the same filepath package, but also the path package as if it were a different package than the one being displayed:

The filepath package uses either forward slashes or backslashes, depending on the operating system. To process paths such as URLs that always use forward slashes regardless of the operating system, see the path package.

What did you expect to see?
I expected to see a package comment which didn't mention non-existent packages.
What did you see instead?
I saw the texts as detailed above.

I've attempted to fix this in https://cuelang.org/cl/1174216.


Mapper always requires FrameAllocator

The trait Mapper<S: PageSize> has a FrameAllocator<Size4Kib> hard-coded in these locations.
https://github.com/rust-osdev/x86_64/blob/master/src/structures/paging/mapper/mod.rs#L188
https://github.com/rust-osdev/x86_64/blob/master/src/structures/paging/mapper/mod.rs#L280
https://github.com/rust-osdev/x86_64/blob/master/src/structures/paging/mapper/mod.rs#L368
I think that should rather be a FrameAllocator<S>, correct me if I'm wrong.

The trait Mapper<S: PageSize> has a FrameAllocator<Size4Kib> hard-coded in these locations.

The FrameAllocator<Size4Kib> is required because mapping a page might require creating new page tables and page tables happen to have a size of 4KiB.

I think that should rather be a FrameAllocator<S>, correct me if I'm wrong.

The methods you listed already take a frame of the appropriate size as a parameter, so there's need to allocate a new one which is why the FrameAllocator<S> bound is not needed.

Ah, got it, thanks!


Fixes #15434: object.d must not import from rt

https://issues.dlang.org/show_bug.cgi?id=15434

The auto-tester fails for some reason.. uhmm. I cannot reproduce on either a 32-bit or a 64-bit linux.

Seems it was a spurious failure, passed now.

I've not seen externDFunc before, interesting! Is that really the only way to declare an external d function? Almost all other runtime extern functions are hooked via C linkage. I'm not sure which way is better.

It is used several times in this file to link to rt functions.

OK, then I guess I just haven't read through it in a while :)
LGTM

@yazd thanks for fixing this!
Btw I wonder how it worked in debug (no inline) mode.
@schveiguy this is a regression from DMD 2.067 (when the code used to work), so I hope this PR will be merged in time for the next release.

@ZombineDev :+1:
I think it works with no inline because the compiler only uses the declaration and links directly to the compiled druntime (it doesn't recompile druntime). I might be wrong.
Is there anything else to address to get this pulled?
Thanks.


Services link in Deploying Challenges tutorial

In the Deploying Challenges section of the tutorial, it says

Login as an administrator and navigate to the Admin Panel.
Click on the “Services” link on the top right

However, when I log into demo.ctfd.io as an admin, I do not see the "Services" link after navigating to the Admin Panel.

Services aren't available on demo at this time. Not really worth the additional overhead and management especially b/c you can always use your own additional infrastructure. If this changes, documentation will be updated.


[CodeCompletion] Don’t increase score for non-default literal types when performing member completion on a literal

Resolves rdar://118999423

@swift-ci Please smoke test

@swift-ci Please smoke test


Fix month view swipe back

I found out that swipe back to February had a bug and would jump to January and looking into it I actually found in full-customization.stories.tsx that the code to move to the previous date on month mode was different so I fixed it using this.

Oh okay, thanks for clarifying that!
On Mon, Feb 14, 2022, 9:57 AM Etienne Maurin @.***>
wrote:

@.**** commented on this pull request.
In src/components/CalendarContainer.tsx
https://github.com/acro5piano/react-native-big-calendar/pull/640#discussion_r805445180
:

   if (mode === 'month') {

     setTargetDate(targetDate.add(targetDate.date() * -1, 'day'))

   } else {

     setTargetDate(targetDate.add(modeToNum(mode, targetDate) * -1, 'day'))

   }

I wanted to but the problem is that modeToNum doesn't know if it's a
swipe forward or back and this condition is only for swipe back.
—
Reply to this email directly, view it on GitHub
https://github.com/acro5piano/react-native-big-calendar/pull/640#discussion_r805445180,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACRZCB77KWO7NS2ETULBXBTU3BHNVANCNFSM5N7VKDMQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you commented.Message ID:
@.***
com>

Released on v3.1.1


Added spec/recipes.

Added Gemfile.
Updated Berksfile 'site' to 'source' per deprecation warning.
Added default_spec.rb to bring ChefSpec::Coverage to 100%
Specs passing.

This looks awesome! Thanks @xacaxulu - this is a :+1: from me.

@xacaxulu Can I ask you to rebase on master? A few things have been fixed in individual commits.

@miketheman I think I correctly fetched upstream and rebased master. Please ping me if I derped it up.

@xacaxulu Looks like a merge, not a rebase. You might want to try git rebase --interactive upstream/master or the like.

Hi @miketheman. If I've sufficiently borked this one up, feel free to decline and I'll submit a new clean PR. My git fu is weak on fork collab stuff :/

@xacaxulu Don't sweat it - I'll rebase and whatnot prior to merging. Thanks!

I pulled this change in with https://github.com/chef-cookbooks/aws/commit/45778269e1a9f5a4c2de538b36bf71ac70c2a7a3
Thanks for helping us add to the testing


Rebuild state serially

We should be rebuilding the state serially, because some operations should not be reordered. I think should read the log in parallel building a sequential list of operations until we reach a stop condition, and then replay these operations sequentially.

Current version is not based on BK


Update app.phy

line 134 Syntax Error: Missing parentheses in call to print.
it should be change like this print(Starting app on port %d % port).

line 134 has some issue


Enable stand-alone conversion without DeepLabCut installed

If accepted this should close  #14.
This PR modifies the code in such a way that the strict dependency on deeplabcut is eliminated. The biggest change is that it moves a small section of code from deeplabcut (the utility to read configuration files, read_config) into this repository. While this might lead to some code duplication in your ecosystem I believe that the benefits or having a less brittle installation pipeline are worth the cost. In order to mitigate this to some degree and avoid code drift I introduced another test which checks that the output of read_config here aligns with the same function on deeplabcut.
Other minor changes I discuss using the code review functionality as I think is easier to read that way.
Let me know if I am missing something.

I was looking at your configuration files (setup.py and setup.cfg) and I did not know what standard do you use for writing your dependencies. In the deeplabcut repo you seem to use a requirements.txt fule but also have them on the setup.py. Let me know what standard you use and I can write the dependencies of the repo there.

setup.cfg is the one :)

setup.cfg is the one :)

@jeylau
I add the dependencies over there then. Also, added deeplabcut to the testing in extras_require.

Just left a minor comment; looking good otherwise, thanks a lot @h-mayorquin! ☺


Can't use GHC >= 7.10

Did anybody try to ask GHC people about this? I can't find any tickets or discussions related. Any references?

Within the first few lines of stack.yaml there is this:
# Unfortunately, ghc versions >= 7.10 currently blow up the type checker when trying to compile samples
# that use more than a few method overloads (Console.WriteLine has heaps hence helloworld cannot be compiled for example)
resolver: ghc-7.8.4

The issue is mainly down to how type families evaluate strictly so that P && Q actually evaluates both sides in many cases. The GHC team knows this and have so far at least put slight hacks in place to help alleviate the problem (note how there's 3 additional equations for both && and || here)
Feel free to try and see if it will compile nicely or if the issue still remains. When I get a chance, I want to experiment with other ways of doing this. In particular, I'm curious to whether a type checker plugin would be the way to go.
As to why it behaved ok on 7.8.4, I'm not too sure but I attempted to get the answer here


Don't mark not required fields as required

Fix #1616 required_if, required_with, etc fields should not be marked as required

Thank you for the fix @mariavilaro ! Merged it, will be available shortly with a composer update, after I bundle it with a few more changes.
Unfortunately, required_if and required_when introduce a level of complexity where a 100% perfect UX solution can't be coded in PHP. Might need some JS. But I don't think it's something we should worry about - I think your solution is good enough for most projects - thank you.


[VideoReader] FFMPEG Multithreading, take2

Addressing issue #3388 on implementing multithreaded video decoding, and superseeding #3389.

Thanks @bjuncek! Could you fix the linter/format errors?

CLosing and sending new one as cleanup


מחשבון אהבה

בהמשך לשיחתנו
Some sort of feature which takes two users (by tag), compares their chat history and interactions, and uses heuristics to determine if they might be a good match as a couple.

@ZeANi-SHILIX אני לא מאמין, זה היה באשמתי? :rofl:  :rofl:  :rofl:

Anyhowwww if you do the Tu B'av random tagging thing again, pick the top MAX(50, 10%) most active users in the chat, who people are most likely to know, and choose random people from that group. Should make it much more interesting than random people from the dregs of the group.


Only request read access to the file when opening the stream.

Since the library is only reading from the stream it can request read-only access to it, which will make it not interfere with other tools reading the credentials file and make it possible to run on read only file systems, such as one shared on a Docker container.

Thanks for making this change and contribute to the library!
Eyal


Brainfuck compatibility

Is this fully Brainfuck compatible? :D

Sadly, it's not - you'll need to wrap your code with 79s so that the compiler recognizes the program as a function; though it still keeps it a valid brainfuck code (79 is just a comment there).
Maybe some day it'll have a full compatibility.


Add dns names to public ip

Add dns names
update client version

Hi @DeepakRajendranMsft, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.
TTYL, AZPRBOT;


export default named function should report top  level name conflict

Describe the bug
export default function x (){}
let x =1;

this code should report that x already defined,  but swc just compiles it as ok
Input code
export default function x (){}
let x =1;

Config
No response
Playground link (or link to the minimal reproduction)
https://play.swc.rs/?version=1.4.8&code=H4sIAAAAAAAAA0utKMgvKlFISU1LLM0pUUgrzUsuyczPU6hQ0NBUqK615spJLQFybBUMrbkAmMVZhSwAAAA%3D&config=H4sIAAAAAAAAA22QOw6DMAyGd06BPLN26daxQw8RpQaFxklkGwmEuHtDCpRK3eL%2F8TnyXNU19GLhWs%2F5mQccE7IjDGr8oWb9hZjulCLrTQRZXQySfeUBS2RpPv1kONvnpkyZNWYF0JIRyy4pNLvby2q1xssvRw13qKUlly0O5IJrpzPcRkqMIjvj4JIJnce%2FaB%2BjnJ2iA8XnUArbIXRK67RuoBh6gW9y%2F8WxEZw89nq5yFK9Abnt2jdaAQAA
SWC Info output
No response
Expected behavior
should report error like below
Identifier 'x' has already been declared. (2:4)

  1 | export default function x(){}
> 2 | let x =1 ;

this error is copied from https://babeljs.io/repl#?browsers=defaults%2C not ie 11%2C not ie_mob 11&build=&builtIns=false&corejs=3.21&spec=false&loose=false&code_lz=KYDwDg9gTgLgBAE2AMwIYFcA29noHYDGMAlhHnCABQCUA3gL4BQmw8IcAvAIxwDcQA&debug=false&forceAllTransforms=false&modules=false&shippedProposals=false&circleciRepo=&evaluate=false&fileSize=false&timeTravel=false&sourceType=module&lineWrap=true&presets=env%2Creact%2Cstage-2&prettier=false&targets=&version=7.24.0&externalPlugins=&assumptions={}
Actual behavior
No response
Version
1.4.8
Additional context
accord to the https://262.ecma-international.org/#prod-ExportDeclaration the export  default function x() {} it should be treated as a function declaration.  but in swc ast it's treated as function expression https://github.com/swc-project/swc/blob/82d1b8e49a242b677d6df259163469cfd1d360c7/crates/swc_ecma_ast/src/module_decl.rs#L197-L205

@kdy1 ，could share some ideas how to fix this，maybe i can make a PR

my first idea is  introducing a new struct like HoistedFnDecl to handle this case, but i will introduce breaking change.
next one's :  in resolve treat export default fn_expr 's  ident as top level variable; then it will be the let a= 1; let a=1 case

You can add visit_export_default_decl to
https://github.com/swc-project/swc/blob/4c83dcc37058495ceeecd3ae225ed95df7cf0b6d/crates/swc_ecma_lints/src/rules/duplicate_bindings.rs#L121-L370 and add binding from there


Freeze in loading.

Trying to run the dapp locally with just npm install and run dev, compile success and run on local host but  it freezes on the loading screen and not processing to the wallet connect page.

Try connecting your local site link locally with Metamask. It is working perfect at all.


auth_uid missing

in line "cookie" part
auth_uid_(NUMBERS)=;",
doesnt print auth_uid, so exported text doesn't work in of-scraper. Tried to log in to OF several times and re-export cookies. Using WIN10 and Firefox 127.0.1 (64-bit)

Your issue is most likely caused by the OF auth changes and not any missing values from the extension (I say that because I just copied the values as-is earlier without any issues).
Check datawhores/OF-Scraper#402 for fixes you can apply to your version of the software to make it work again.
If you run a recent enough version, you should be able to just update your config.json for of-scraper.


[feat]: add group prop

This PR implements the group prop using an event bus instead of component references.
Checklist

[x] Implement group prop.
[x] Update docs.

Overview
Carousels can now be synced no matter how deeply nested inside other components, it also supports binding the group prop to get dynamic groups working.
This PR addresses the goals in #94 :

Ability to sync more than two carousels.
Syncing carousels will be context independent.
Fix issue when passing carousel to another component slot.

Example usage:
<hoooper group="myCarousel">
    ...
</hooper>

<hoooper group="myCarousel">
    ...
</hooper>

<hoooper group="myCarousel">
    ...
</hooper>

PS: This PR removes some unused dependencies due to Vuepress 1.x upgrade.
Issues affected
closes #94
closes #69
closes #55

It looks like you are using a different lint setup (using double quotes, and space after function declaration). So please include your lint setting file, to have a consistent style all over the project.

@Abdelrahman3D I have updated the linting config as requested.


Add pyproject.toml in place of setup.py/cfg

This does an initial basic replacement of setup.py / setup.cfg to use pyproject.toml.
As mentioned here, there was a block due to using lib3to6, but as python 2.x is end of life'd... i've just removed sub 3.6 dependencies.
I also replaced the deprecated pkg_resources with packaging.version.parse.
This breaks a couple of CLI tests partly due to logging differences, which i'm not sure about.

Hi @jgrey4296,
is there an issue with the current approach? What problem does this solve for an end-user?
Cheers Manuel

less an issue, just standardization. with 2.7 EOL, the additional dependencies and older build config were extra hurdles to getting to what I wanted: whitespace patterns.
I ended up doing this shift at the same time as I was figuring out the project code. if you want to use it, cool, if not, cool.

Why
https://setuptools.pypa.io/en/latest/userguide/quickstart.html#basic-use

We recommend users expose as much as possible configuration in a more declarative way via the pyproject.toml or setup.cfg, and keep the setup.py minimal with only the dynamic parts (or even omit it completely if applicable).

From validate-pyproject README:

With the approval of PEP 517 and PEP 518, the Python community shifted towards a strong focus on standardization for packaging software, which allows more freedom when choosing tools during development and make sure packages created using different technologies can interoperate without the need for custom installation procedures.

This shift became even more clear when PEP 621 was also approved, as a standardized way of specifying project metadata and dependencies.

validate-pyproject was born in this context, with the mission of validating pyproject.toml files, and making sure they are compliant with the standards and PEPs.


Pull in the latest tf-apko, apko, tf-oci

Fixes: https://github.com/chainguard-images/images/pull/846

Waiting on downstream PR to check things before this is RFAL


How to stop Dialog from closing when touching outside the box?

Same way you do it with an AlertDialog. You can use cancelable(false) on the Builder or setCancelable on the MaterialDialog.

Thank you.