Bimodal behavior of CCFMNRV in C channel in recent upsAnd series (C-channel SNR=400)

Observations of Aug 14 and 18 of two long OG for Stefan Pelletier on upsAnd show weird behavior of the FP drift. The plots below show CCFMNRV versus time for the full night Aug 14 and 18.
Other transits or long OG observed this run do not show this behavior, SNR of the FP is usualy always lower.
I thought it was maybe related to the number of FP peaks extracted by the DRS, maybe an order where the number of peaks is switching between 2 values +/- randomly during the series. I tought it would change the wavesolution for that order and end in that 1.7m/s unstability.   I checked all log files APEROcal_extractlog  but didn't find anything:  for a given order 0-48 the number of peaks is the same for all exposures.
I'll explore the ccf logs also again.
Below are 2 plots for each night showing the issue with CCFMNRV and the measured SNR in the C-channel

After the current run, I can do a test with short series of FP on AB and C with different SNR to see the SNR at which the issue with CCFMNRV occurs.

The SNR is in which order? Do I read this graph as the SNR is exactly 400 everywhere or is the value of SNR > 400?
I don't think you can blame the wave solution for this - there should only be 2 wave solutions per night - one for the PM calibration and one for the AM calibration - it seems your oscillation here is within a night?  To be sure check which wave solution was used with each file for 59455.40 to 59445.60 - if they use one wave solution - then the wave solution is constant (so it cannot be a problem) if they use two (the PM and the AM) you could only see a jump from one to the other - the only way you could blame the wave solution is if it jumped from one wave solution to the other following your oscillation - I cannot see any way this could happen in the DRS.
You should probably isolate the oscillating data and plot some of the orders in the extracted spectrum:
i.e. plot order the e2dsff for order 35 for the observations from 59445.40 to 59445.60 and see if you can see any change.
Note that extraction is simple (not much from the drs can go wrong) it just takes the pixels in the location and sums them - if there is something wrong there it is most likely something wrong with the raw data itself (i.e. not a DRS problem).
If I recall correctly this star (upsAnd) is extremely bright, correct? the FP ND is normally adjusted to match the brightness of the star correct? This would explain your high SNR FPs? If I had to blame anything it would be that - maybe the FP are saturated or pushing the FP that bright means the FP peaks do not look "normal" the ND adjustment just doesn't work for this magnitude? - I think plotting the e2dsff for "normal" and your "oscillating" cases will show you interesting things and maybe explain it or at least show you weird things happening - if they look completely "normal" then maybe it is a drs issue.

hi Luc, I've seen that the FP neutral density value is 0.82 for this sequence. It is in a range where I don't know the behaviour of the density. It is not a problem of saturation, because the SNR is not that large, in my opinion, but it could be that the ND has weird behavior at values <1 where it's never used... I would test that first...

Hi Neil  The SNR is extracted from order 34 (Y axis legend).  As Claire says, it is not saturating, but clearly the issue seems to be connected to the relatively high SNR in the FP C-channel.    yes the star is bright, set to H=3 in the observing tools (but it may be even brighter, as PI cannot enter H<3)  I'll have a look at the e2ds as you suggest

Hi Claire  testing the ND means doing the test I was mentioning in the comment I added yesterday, i.e. "short series of FP on AB and C with different SNR to see the SNR at which the issue with CCFMNRV occurs."

For the record: AB has SNR=600.

Hi Neil
You are correct about the WAVEFILE:  the only change occurs at midnight HST, and this leads to the -1.5m/s jump at 59441.418.
dfits -x 1 *o_pp_e2dsff_C_ccf_smart_fp_mask_C.fits |fitsort MJDATE HSTTIME CCFMNRV WAVEFILE WAVETIME OBJECT PI_NAME |grep Pelletier

I had a look at the fourth extension of the o.files,  many peaks in the H band do not reach the 14 reads, the brightest peaks in H seem to reach 8 counts before saturation.  I didn't try to see if these numbers fluctuate for a given pixel in the time series, but I can do it quite easily.  If I see fluctuation between 2 values, the slope would oscillate between 2 values too, but I don't see yet how it could lead to a bimodal behaviour of the drift.   I can extract an image for only say 7 reads, and see if the C channel behaves correctly. That's your idea @eartigau ?

Having FP peaks that do not reach the number of reads in a frame is a very bad idea. You do not improve the photon noise and you will most likely have 2nd order effects where there is a coupling between an imperfect correction of linearity and RV. Do you see pixels on the science traces that do not reach the 14 reads? Can you check the median value of the intercept? That’s 3rd extension. If it fluctuates, then it adds a DC level to the slope fitting and the number of valid reads. I’m also a bit puzzled as to how this could translate into a bimodal distribution.

Le 3 sept. 2021 à 00:19, larnoldgithub @.***> a écrit :
I had a look at the fourth extension of the o.files, many peaks in the H band do not reach the 14 reads, the brightest peaks in H seem to reach 8 counts before saturation. I didn't try to see if these numbers fluctuate for a given pixel in the time series, but I can do it quite easily. If I see fluctuation between 2 values, the slope would oscillate between 2 values too, but I don't see yet how it could lead to a bimodal behaviour of the drift. I can extract an image for only say 7 reads, and see if the C channel behaves correctly. That's your idea @eartigau https://github.com/eartigau ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/njcuk9999/apero-drs/issues/672#issuecomment-912243637, or unsubscribe https://github.com/notifications/unsubscribe-auth/AIDD7EENDBXGMJRSKUA677LUABEGZANCNFSM5CTXG2WA.
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.

I plotted the Yintercept of a random point with NREADS=14 (nominal) and there is a bimodal distribution of the Yintercept:

for both pixels, the variations are phased, only amplitude and mean value change.
Not all pixels with the nominal NREADS=14 shows this bimodal behavior:

on the plot below: Yintercept, slope and nreads for a given pixel. Only the Yintercept is bimodal.


Make favorites icon not the moon icon

how do i enable this favorites thing

send discord://-/channels/@favorites in a channel and click on it, it's icon will appear below the dm icon

actually this might require the 2021-09_favorites_server experiment to be enabled and you to add a channel/category in the favorites server for the icon to show up.
i'm using Stable 245033 (8b6a6af) Vencord 96126fa (Standalone)Electron 22.3.26Chromium 108.0.5359.215Host 0.0.283 arm64 OS X 10.15.7 (23.1.0)
With my patch:

Without my patch:

Doesn't work for me on vesktop. I will merge later once I can verify this is fine.

I have fixed this in 898a02b with a different method. Thanks for pointing it out.

i just realized your way is better lol


Use OPERATOR_IMAGE_URI and add CATALOG_IMAGE_URI

Take advantage of https://github.com/openshift/dedicated-admin-operator/pull/27

/hold until #12 please


Publish refactor extra

Do not consider this, just for testing things related to https://github.com/atlassian/changesets/pull/674#issuecomment-982022395. I'll close after

Closed as it does it's job. sorry for the noise


BUG: Move custom icons to wowchemy-dictated location

Move custom icons to wowchemy-dictated location.
Reference documentation:
https://wowchemy.com/docs/getting-started/page-builder/#icons

Related to:
https://github.com/brainhackorg/global2021/issues/76
and cross-referencing https://github.com/Remi-Gau/global2021/pull/19#issuecomment-999245476
Run out of ideas if this does not make the trick.
@Remi-Gau


RC Spot-Check Test: Energy Skate Park 1.3.0-rc.2

RC Spot-Check Test
Mentions: @marlitas @amanda-phet @Luisav1
Simulation links

github repository for issues
phet top-level directory
sim: all_phet.html

Test Matrices
General Test:

[x] Tester = NS , Platform = mac + safari , Time = .5
[ ] Tester = , Platform = , Time =

Features included

[ ] PhET-iO
[ ] Alternative Input
[ ] UI Sound
[ ] Sonification
[ ] Description
[ ] Voicing

Focus and Special Instructions
This spotcheck includes minor changes to the prior RC:

Adding a label and description to the language selection in preferences.
Consistent designs in clothing when characters change direction.
Preferences dialog text now stretches with ?stringTest=double.

Issues to Verify
The issues can be closed after verification unless stated otherwise.

[x] https://github.com/phetsims/energy-skate-park/issues/376
[x] https://github.com/phetsims/energy-skate-park/issues/375
[x] https://github.com/phetsims/joist/issues/945 (Please comment in issue on verification status for this RC)

For QA...

General features

What to Test

Click every single button.
Test all possible forms of input.

Test all mouse/trackpad inputs.
Test all touchscreen inputs.

If there is sound, make sure it works.
Make sure you can't lose anything.
Play with the sim normally.
Try to break the sim.
Test some query parameters. (See QA Book for a
list of query parameters.)
When making an issue, check to see if it was in a previously published version.
Try to include version numbers for browsers.
If there is a console available, check for errors and include them in the Problem Description.
As an RC begins and ends, check the sim repo. If there is a maintenance issue, check it and notify developers if there
is a problem.
As the RC ends, notify the developer of any new QA credits that need to be added.

PhET-iO features

What to Test

Make sure that public files do not have password protection. Use a private browser for this.
Make sure that private files do have password protection. Use a private browser for this.
Make sure standalone sim is working properly.
Make sure the wrapper index is working properly.
Make sure each wrapper is working properly.
Launch the simulation in Studio with ?stringTest=xss and make sure the sim doesn't navigate to youtube
For newer PhET-iO wrapper indices, save the "basic example of a functional wrapper" as a .html file and open it. Make
sure the simulation loads without crashing or throwing errors.
Load the login wrapper just to make sure it works. Do so by adding this link from the sim deployed root:/wrappers/login/?wrapper=record&validationRule=validateDigits&&numberOfDigits=5&promptText=ENTER_A_5_DIGIT_NUMBER

Further instructions in QA Book
Conduct a recording test to Metacog, further instructions in the QA Book. Do this for iPadOS + Safari and one other
random platform.
Conduct a memory test on the stand alone sim wrapper (rc.1).
Test one platform combination with ?phetioDebug=true on the Studio and State wrapper.
If Pan/Zoom is supported, make sure that it works when set with PhET-iO State.
Test that the sim works offline:

Click the link to the phet-io zip file (at top of issue) to download the zip file.
Unzip it to a spot locally.
Open index.html by double clicking it on your desktop or in a Finder-view.
It should look like the standalone version of the sim in PhET-iO brand.

Accessibility features

What to Test

Specific instructions can be found above.

Make sure the accessibility (a11y) feature that is being tested doesn't negatively affect the sim in any way. Here is
a list of features that may be supported in this test:

Alternative Input
Interactive Description
Sound and Sonification
Pan and Zoom
Mobile Description
Voicing

Test all possible forms of input.

Test all mouse/trackpad inputs.
Test all touchscreen inputs.
Test all keyboard navigation inputs (if applicable).
Test all forms of input with a screen reader (if applicable).

If this sim is not in this list or up to date there, make an
issue in website to ask if PhET research page links need updating. Please
assign to @terracoda and @emily-phet.

Screen Readers
This sim may support screen readers. If you are unfamiliar with screen readers, please ask Katie to introduce you to
screen readers. If you simply need a refresher on screen readers, please consult the
QA Book, which should have all of the information
you need as well as a link to a screen reader tutorial made by Jesse. Otherwise, look over the a11y view before opening
the simulation. Once you've done that, open the simulation and make sure alerts and descriptions work as intended.

Platforms and Screen Readers to Be Tested

Windows 10 + Latest Chrome + Latest JAWS
Windows 10 + Latest Firefox + Latest NVDA
macOS + Safari + VoiceOver
iOS + Safari + VoiceOver (only if specified in testing issue)

Critical Screen Reader Information
We are tracking known screen reader bugs in
here. If you find a screen reader bug,
please check it against this list.

Keyboard Navigation
This sim supports keyboard navigation. Please make sure it works as intended on all platforms by itself and with a
screen reader.
Magnification
This sim supports magnification with pinch and drag gestures on touch screens, keyboard shortcuts, and mouse/wheel
controls. Please test magnfication and make sure it is working as intended and well with the use cases of the
simulation. Due to the way screen readers handle user input, magnification is NOT expected to work while using a screen
reader so there is no need to test this case.

FAQs for QA Members

There are multiple tests in this issue... Which test should I do first?
Test in order! Test the first thing first, the second thing second, and so on.

How should I format my issue?

We typically assign the developer who opened the issue in the QA repository.

My question isn't in here... What should I do?
You should:

Consult the QA Book.
Google it.
Ask Katie.
Ask a developer.
Google it again.
Cry.

Shouldn't Luisa be added to team credits?
QA is done.

Closing since the next spotcheck for ESP is in the pipeline.


Not worked with vuejs 3

It does not worked with vuejs 3.
How to make this options="hospitals"  to like this
<select class="form-control" v-model="patient.hospital_id" name="hospital_id" required >
  <option value="" selected>Select Hospital</option>
  <option v-for="hos in hospitals" :key="hos.id" :value="hos.id">{{ hos.name }}</option>
</select>

where hospitals  = [ { "id": 1, "name": "BPKIHS" }, { "id": 2, "name": "Dhulabari Help post" } ] - more object of array.
I wish it worked like bootstrap select.

Do u read docs? This package works great with vue3

Does placeholder works for you?

If not, maybe you can try the following workaround:
https://codesandbox.io/s/vue-next-select-issue-218-7d48y?file=/src/App.vue

Prepend an option like your example:

const options = ref([
  { id: "", name: "Select Hospital" },
  // ... other options here
]);

Make modelValue always be first option if there is no selected option

watchEffect(() => {
  if (modelValue.value === null) {
    modelValue.value = "";
  }
});

Set min to 1 to prevent de-select (but allow to select 'Select Hospital' option) like html select:

<vue-select
  :min="1"
  ... other props here
/>

Does placeholder works for you?
If not, maybe you can try the following workaround:
https://codesandbox.io/s/vue-next-select-issue-218-7d48y?file=/src/App.vue

Prepend an option like your example:

const options = ref([
  { id: "", name: "Select Hospital" },
  // ... other options goes here
]);

Make modelValue always be first option if there is no selected option

watchEffect(() => {
  if (modelValue.value === null) {
    modelValue.value = "";
  }
});

Set min to 1 to prevent de-select (but allow to select 'Select Hospital' option) like html select:

<vue-select
  :min="1"
  ... other props goes here
/>

I am using option API in vue.js3 not composition api

@bhojkamal  I have changed the sandbox to option API and it woks, can you provide more details?

<template>
      .....
Select Hospital: *
<select class="form-control" v-model="patient.hospital_id" name="hospital_id" required >
            <option value="" selected>Select Hospital</option>
            <option v-for="hos in hospitals" :key="hos.id" :value="hos.id">{{ hos.name }}</option>
          </select>
      .......
</template>
And the option value come from ajax. like this -

export default {
  data() {
      return {
          patient: {
            hospital_id:'',
          },
          hospitals:{},
      }
  },

created() {
this.$axios.get('/api/users/getHospital')
            .then(response => {
                this.hospitals = response.data;
            })
            .catch(function (error) {
                console.error(error);
            });
   },
addTelePatient() {
//
}
}

please see below.

Everything works fine:
<template>
  <vue-select
    :disabled="loading"
    :loading="loading"
    v-model="modelValue"
    :options="normalizedOptions"
    label-by="name"
    value-by="id"
    :min="1"
  />
</template>

I'll gonna close this issue.

@iendeavor I get value from server, it is not static value.

The await is to simulate getting the value from server.
async fetchData() {
  this.loading = true;

+ await new Promise((resolve) => setTimeout(resolve, 1000));
  this.options = [
    { id: 1, name: "BPKIHS" },
    { id: 2, name: "Dhulabari Help post" },
  ];

  this.loading = false;
},

Could you help on this please?

can't we type and filter from the list? I have not seen on the code sandbox. It is normal on select options to type and filter quickly.


Perf (implementation 2): in iac diff scan, allow file filter to operate without opening the files

This attempts to solve the same issue as #602.
This time, we provide two distinct acceptation functions as arguments to the tar_from_ref_and_filepaths function:

One filtering on filepath only (does not open the file)
One filtering on file content. The file is opened once. The filepath is also provided as an argument, because cross-validation on path and content might be needed in the future.

Pros:

When validating on path only, there is no longer an unused file_opener argument in the acceptation function
Cons:
However, the path argument in the second acceptation function might be unused.

Codecov Report

Merging #606 (8c13803) into pbes/IAC-95/iac-diff-only-read-file-if-needed (2f69199) will decrease coverage by 0.03%.
The diff coverage is 100.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@                                Coverage Diff                                @@
##           pbes/IAC-95/iac-diff-only-read-file-if-needed     #606      +/-   ##
=================================================================================
- Coverage                                          93.82%   93.80%   -0.03%
=================================================================================
  Files                                                104      104
  Lines                                               4856     4856
=================================================================================
- Hits                                                4556     4555       -1
- Misses                                               300      301       +1

Flag
Coverage Δ

unittests
93.80% <100.00%> (-0.03%)
:arrow_down:

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

Impacted Files
Coverage Δ

ggshield/cmd/iac/scan/iac_scan_utils.py
94.11% <100.00%> (ø)

ggshield/core/git_shell.py
92.20% <100.00%> (-0.65%)
:arrow_down:


[Documentation] Installing in Mac Catalina or higher

The problem
I had a problem installing flutter at my Macbook Air 2017 (OS version = Big Sur).
Step by step to reproduce

After doing exactly equal as the installing documentation, I restarted the computer
tried to run flutter doctor
Terminal was logging command not found

The Solution

I googled some solutions and I found that Mac Catalina or Higher requires Full Disk access to Terminal to get Flutter working, but I have no evidence it works (Anyway, I allowed this in preferences).
I found out that Mac Catalina or higher uses zsh with a .zprofile (not .zshrc nor .zshenv etc).
Then I added this to the .zprofile:
export PATH=$HOME/FLUTTER_DIR/flutter/bin:$PATH

AND..... IT WORKS!
Proposal
Consider adding a note at Get Started Documentation.
Best regards.

Hi @danielhrc
For a beginner, I understand this could be confusing especially if you're new to the Apple environment.
To propose this update in the getting started page, please file an issue on flutter/website repository
Thank you


Tune Decoupled Look-back based Algorithms for H100

After https://github.com/NVIDIA/cub/pull/710 is merged, we should tune the following algorithms:
### Tasks
- [x] Introduce SM90 tuning policy into scan
- [x] Introduce SM90 tuning policy into select
- [x] Introduce SM90 tuning policy into partition
- [x] Introduce SM90 tuning policy into scan by key
- [x] Introduce SM90 tuning policy into reduce by key
- [x] Introduce SM90 tuning policy into unique by key
- [x] Introduce SM90 tuning policy into three-way partition
- [ ] Introduce SM90 tuning policy into batched memcpy
- [x] Introduce SM90 tuning policy into run length encode

Extracted batched memcpy tuning into https://github.com/NVIDIA/cccl/issues/237.


Log nodes only supports nodes 1-232 even though Long Range Nodes start at 256

Checklist

[ ]  I am not using Home Assistant. Or: a developer has told me to come here.
[X] I have checked the troubleshooting section and my problem is not described there.
[X] I have read the changelog and my problem is not mentioned there.

Deploy method
Home Assistant Add-on
Z-Wave JS UI version
9.16.2
ZwaveJS version
13.0.2
Describe the bug
Settings -> Z-wave -> Log nodes only supports nodes 1-232 even though higher node numbers are possible due to Z-Wave Long Range. This means you cannot restrict logging to long range nodes.
To Reproduce

Have a Z-wave stick/dongle/whatever that allows for Long Range Devices and connect at least one
Go to Settings
Z-wave Dropdown
"Log nodes" Setting
Type in the ID of any Z-wave Long Range Node
Gives error "Nodes must be between 1-232", attempting to save gives error "Your configuration contains errors, fix it"

Expected behavior

Have a Z-wave stick/dongle/whatever that allows for Long Range Devices and connect at least one
Go to Settings
Z-wave Dropdown
"Log nodes" Setting
Type in the ID of any Z-wave Long Range Node
No error, saving works

Additional context
I am using Home Assistant bum bum bum. However, this is most probably likely not a Home Assistant issue. If this is a Home Assistant issue, feel free to let me know and I will happily move it to another repository.

@AlCalzone Should I change the check so?


export a target dossier

Closing pending fresh requests from partners based upon their feedback on the plans for new features and data and POC work ongoing for Target profile page.


modals: Adjust copy, layout of move/rename modals.

CZO discussion

Screenshots and screen captures:

Before
After

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[x] Calls out remaining decisions and concerns.
[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[x] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

Maybe s/Rename topic/Rename/ in that second screenshot would be nicer wording?

I did some language tweaks after discussion with Alya, but don't see the CSS code for changing the wrapping behavior, and it doesn't wrap in my development environment, so leaving for you to take a next look. Feel free to DM me to merge once you've settled this; I'd like to get it on chat.zulip.org ASAP for testing.

I guess after some more thinking, I think we might as well merge this and that can just be a next PR; it's definitely better than the previous version.

To clarify the comments above, @timabbott tell me we still need a PR to change the title 1 line -> 2 lines.


Sharing user link: pin is not visible

When sharing a user link the map should zoom to a level where the user can be focused on.
Using the filter center focuses on the 0,0 point on the map, which is also wrong

Done in #83


Changing a single file in the engine sometimes cause scons to build tons of other files

Godot version:
Godot 3.2 stable
OS/device including version:
Ubuntu 18.4.4 LTS
Issue description:
When I change a single file in the engine (for example file_access.cpp) and recompile it also compiles other stuff, image:

I checked that I did not changed the code with git (no pull, no branch, no checkout... so only I changed the code)
Steps to reproduce:
It happens only sometimes
Usually happens when 2+ minutes passed or something like that from last compile
Minimal reproduction project:

That's normal, you're change file_access.h so any file that includes it, and any file that includes those files, etc., needs to be recompiled.


First Draft

Codecov Report

Merging #174 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #174   +/-   ##
=======================================
  Coverage   60.19%   60.19%
=======================================
  Files           6        6
  Lines         314      314
=======================================
  Hits          189      189
  Misses        125      125

Flag
Coverage Δ

#unittests
60.19% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out 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 0259e00...2efc534. Read the comment docs.

Ok take a look - I think the intro still focuses too much on heterogeneity but I left it in as I want to talk to Dr. Meyer about it before we really reprogram the DNA of that opening paragraph. I also just shortened some language in the discussion as reading through it made me feel as though most of the points were somewhat vital.
Also I'm going to open up an issue about the sigma point stuff as I think your issues with the mixture sigma point stuff and the optimization of R1hi R2hi in fig 6 tell us that somethings screwy with it.

Tried to figure out how to add a review on a closed branch lol. I looked it through. Other than the points we talked about, it seems like most edits just provide very minor improvement. This is a good sign that the manuscript is in a good shape now. I’ll check on the issue you raise, and let’s see if this changes the results in the optimization.


Calling IMMDevice.Activate() results in a NullReferenceException

Actual behavior
Calling the IMMDevice.Activate() overload with the convenience extension (e.g. the one that takes a Guid rather than a Guid*) method results in a NullReferenceExcception.
It seems the issue is in the generated convenience extension method that looks like this:
internal static unsafe void Activate(this winmdroot.Media.Audio.IMMDevice @this, in global::System.Guid iid, winmdroot.System.Com.CLSCTX dwClsCtx, winmdroot.System.Com.StructuredStorage.PROPVARIANT ? pActivationParams, out object ppInterface)
{
	fixed (global::System.Guid* iidLocal = &iid)
	{
		winmdroot.System.Com.StructuredStorage.PROPVARIANT pActivationParamsLocal = pActivationParams ?? default(winmdroot.System.Com.StructuredStorage.PROPVARIANT );
		@this.Activate(iidLocal, dwClsCtx, pActivationParams.HasValue ? pActivationParamsLocal : Unsafe.NullRef<winmdroot.System.Com.StructuredStorage.PROPVARIANT >(), out ppInterface);
	}
}

The actual culprit is the Unsafe.NullRef<PROPVARIANT>() call--it seems that passing a non-nullable struct type to it results in a NullReferenceException.
Expected behavior
Calling the .Activate() convenience overload works!
Repro steps

NativeMethods.txt content:

CoCreateInstance
MMDevice
MMDeviceEnumerator
IMMDeviceEnumerator
IAudioMeterInformation

NativeMethods.json content (if present):
N/A

Any of your own code that should be shared?

Here's a little minimal sample, cut down a bit from what I have:
IAudioMeterInformation _audioMeterInfo;
HRESULT devEnumResult = PInvoke.CoCreateInstance(
    typeof(MMDeviceEnumerator).GUID,
    null,
    CLSCTX.CLSCTX_INPROC_SERVER,
    typeof(IMMDeviceEnumerator).GUID,
    out object rawDeviceEnumerator);

if (devEnumResult.Failed)
{
    Debug.WriteLine($"Failed to create a device enumerator: HRESULT {devEnumResult}");
    return;
}

IMMDeviceEnumerator? deviceEnumerator = (IMMDeviceEnumerator)rawDeviceEnumerator;
deviceEnumerator.GetDefaultAudioEndpoint(EDataFlow.eRender, ERole.eConsole, out IMMDevice mmDevice);

// this following block *does* work.
unsafe
{
    Guid infoGuid = typeof(IAudioMeterInformation).GUID;
    Guid* iidLocal = &infoGuid;
    var nullPropvariant = new PROPVARIANT();
    mmDevice.Activate(iidLocal, CLSCTX.CLSCTX_ALL, nullPropvariant, out object rawMeterInfo);
    _audioMeterInfo = (IAudioMeterInformation)rawMeterInfo;
}

//mmDevice.Activate(typeof(IAudioMeterInformation).GUID, CLSCTX.CLSCTX_ALL, null, out object rawMeterInfo); // <-- this crashes!

Context
Note: this is a WPF project.

CsWin32 version: 0.3.49-beta
Win32Metadata version (if explicitly set by project): Not explicitly set
Target Framework: net7.0-windows
LangVersion (if explicitly set by project): [e.g. 9]: Not explicitly set

I hit this as well. The nullable pActivationParams isn't being handled correctly here, ends up trying to access pActivationParams.HasValue.
Poking @AArnott for prioritization 😅

I don't think there's anything wrong with calling pActivationParams.HasValue in the friendly overload, as that is a nullable struct, so it can't throw NRE.
I suspect the issue is that the .NET interop layer cannot handle a null reference passed into an in struct parameter on a COM interface. That seems unfortunate because the COM side should only need the address, and taking the address of a null reference is allowed in C# and produces a null pointer as intended.
But on the flip side, I don't know why CsWin32 is generating the COM interface with an in PROPVARIANT rather than PROPVARIANT* in the first place. Given it's an [Optional] parameter, CsWin32 should be (I think) be preferring pointers. That's the first thing I'll look into.

This bug was surprisingly difficult to solve.

Yay, thanks @AArnott. Will try it out tonight.


Only consider directories

Is your feature request related to a problem? Please describe.
It seems like dropignore scans all directories and files and tests the pattern against all of them. If I only need to ignore directories, it would probably be a good performance optimization to only check the pattern against directories.
Describe the solution you'd like
An additional config option that instructs dropignore to only consider directories, for example:
matcher:
  ignore-specs:
    - pattern: "**/node_modules"
      type: "d"

Describe alternatives you've considered
I tried using the pattern "**/node_modules/" (with a trailing /) but that didn't match anything.
Additional context
My Dropbox folder contains ten times as many files as directories:
> find ./Dropbox -type f | wc -l
445649

> find ./Dropbox -type d | wc -l
44862

Thanks for the detailed request @Hubro.
While scanning (file system traversal) or watching (file system notifications) the app handles the file system entries it is being passed. At that point in time it doesn't know if that file system entry is a file, a directory or a symlink. Currently the code does not retrieve this meta information from the file system (an additional IO operation) but just matches the path the file system entry carries as a property which does not need to be retrieved again.
Some simple testing with 533k entries (452k files) and warmed up caches shows a small penalty on additionally querying the file system entry type.
IMHO the matching of the path against the pattern needs to be done in any case because there might be other patterns defined which might target files.
I still think the proposal is useful to have ignore or skip specifications target only specific file system entry types. E.g. ignore a file foo, but not the directory foo. Actually, the matcher spec entries are objects and not just a set of strings for easier extensibility because I saw this coming one day. But even then, the path matching would very likely always be done, and only if the type property is not "any" (the default) the file systems entry type would be queried. (To safe on the expensive IO operation when not needed.)
Nevertheless, would you agree - with the explanations above - your desired goal of saving some IO operations while scanning and watching is not really possible. (Basically the traversal and file system notifications operate on file system entries no matter if the entry represents something you are interested in or not.)

Makes sense to me :slightly_smiling_face: Thanks for the detailed response.

Thanks for the feedback. I have been writing a bit cryptic yesterday evening. I just want to clarify ...
On initial scanning/traversal, whenever a file system entry is encountered which matches and is by chance a directory - e.g. node_modules - it is not traversed any further, so the thousands of files in that folder are not traversed. What is happening in the watch phase though - and that leaves room for optimization - is that when a file system entry is modified in an ignored directory it's not gone by unhandled but evaluated as any other file system entry allthough this would not be needed as the partent directory is ignored. It's not that of a showstopper but I will keep an eye on it.
Happy the app is useful to you.

That's good to know, thanks :slightly_smiling_face:


Updated validations.py script

Reversed boolean values of final conditional of validate_user function in validations.py
Closes:#1
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

@googlebot I signed it!


samples: immediate logging  mode in samples nvs

Do not direct printk to logging subsystem in the sample.
This PR is similar to https://github.com/zephyrproject-rtos/zephyr/pull/52949/
It makes

samples/drivers/watchdog : fixed by https://github.com/zephyrproject-rtos/zephyr/pull/52949/
samples/subsys/task_wdt : fixed by https://github.com/zephyrproject-rtos/zephyr/pull/52949/
samples/subsy/nvs
PASSED

Signed-off-by: Francois Ramu<EMAIL_ADDRESS>

For my understanding: Why is this required?
In the linked PR the argument was that in a test that tests the logging subsystem itself it should not be used by the application for other purposes (printk messages) at the same time. I don't see this being the case for watchdog and NVS.

Changing printk with printf in src/main.c sample application instead  makes the testcases PASSED as well

samples PASSED with twister without CONFIG_LOG=y

and  w/o CONFIG_WDT_LOG_LEVEL_DBG=y in the samples/subsys/task_wdt/prj.conf
and  w/o CONFIG_NVS_LOG_LEVEL_DBG=y in the  samples/subsys/nvs/prj.conf
and  w/o CONFIG_WDT_LOG_LEVEL_DBG=y in the samples/drivers/watchdog/prj.conf

Still failing : zephyr/samples/subsys/nvs/sample.nvs.basic :
Rebooting in ...561 62 --- 115 messages dropped ---
63 64 65 66 67 68 69 6a 6b 6c 6d 6e 6f 70 71 72 73 74 75 76 77 78 79 7a 7b 7c 7d 7e 7f
...28--- 79 messages dropped ---
...27...26...25...24...23...22...21...20...19...18...17...16...15...14...13...12...11...10...9...8...7...6...5...4...3...2...1...0
Oldest reboot counter: 0
Rebooting in ...561 62 --- 115 messages dropped ---
63 64 65 66 67 68 69 6a 6b 6c 6d 6e 6f 70 71 72 73 74 75 76 77 78 79 7a 7b 7c 7d 7e 7f
...28--- 80 messages dropped ---
...27...26...25...24...23...22...21...20...19...18...17...16...15...14...13...12...11...10...9...8...7...6...5...4...3...2...1...0
Oldest reboot counter: 0
Rebooting in ...561 62 --- 115 messages dropped ---
63 64 65 66 67 68 69 6a 6b 6c 6d 6e 6f 70 71 72 73 74 75 76 77 78 79 7a 7b 7c 7d 7e 7f
...28--- 81 messages dropped ---
...27...26...25...24...23...22...21...20...19...18...17...16...15...14...13...12...11...10...9...8...7...6...5...4...3...2...1...0
Oldest reboot counter: 0
Rebooting in ...561 62 --- 115 messages dropped ---
63 64 65 66 67 68 69 6a 6b 6c 6d 6e 6f 70 71 72 73 74 75 76 77 78 79 7a 7b 7c 7d 7e 7f
...28--- 82 messages dropped ---
...27...26...25...24...23...22...21...20...19...18...17...16...15...14...13...12...11...10...9...8...7...6...5...4...3...2...1...0

Change the prj.conf  with  LOG_MODE_IMMEDIATE to have the zephyr/samples/subsys/nvs/sample.nvs.basic PASSED:
*** Booting Zephyr OS build zephyr-v3.2.0-2493-gfe7ab0939d54 ***
[00:00:00.005,000]  fs_nvs: nvs_recover_last_ate: Recovering last ate from sector 0
[00:00:00.016,000]  fs_nvs: nvs_add_gc_done_ate: Adding gc done ate at 3f0
[00:00:00.024,000]  fs_nvs: 3 Sectors of 1024 bytes
[00:00:00.030,000]  fs_nvs: alloc wra: 0, 3e8
[00:00:00.035,000]  fs_nvs: data wra: 0, 0
No address found, adding <IP_ADDRESS> at id 1
No key found, adding it at id 2
No Reboot counter found, adding it at id 3
Id: 4 not found, adding it
Longarray not found, adding it as id 5
Reboot counter history: ...0
Oldest reboot counter: 0
Rebooting in ...5...4...3...2...1


Extend TopoManager to work on an n-dimensional torus

Original author: Nikhil Jain
Original issue: https://charm.cs.illinois.edu/redmine/issues/19

Extend the TopoManager API to work on any n-dimensional torus.

Original date: 2013-03-14 22:51:15

API changes and bugfix releases don't go together. Save it for 6.6.


⚠ Bofrost.lu has degraded performance

In ba6d536, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1174 ms

Resolved: Bofrost.lu performance has improved in 1e90a3e.


*: refactor agent and add config controller

Please do not review yet lots of things changing.
Built on #88

Overall, this refactoring looks great!
One question I still have is why introduce parallelism that makes it harder to reason about the agent to ensure determinism and safety? We're like polling for status update at most 1/min while taking ms to complete, so it's not clear why we need a worker queue and async exporters/observers?

I need to rebase


tumblr_nbt7suBKo71tlr3u4o1_1280 caption fail

caption of video tumblr_nbt7suBKo71tlr3u4o1_1280 is Fail in https://github.com/raingo/TGIF-Release/blob/master/data/tgif-v1.0.tsv . I would caption as a man is playing guitar with a light flashing behind him. How it happens and could you fix this in your dataset?

Thanks for reporting.
This has been fixed in 3e54d2f71418d8a2e9f5f61aa5be0edb9c0ac2b8, and the right caption A guy in red shirt playing guitar. is in place now.
Please feel free to reopen if there are similar problems.


Generated code of init task should ensure separation from existing code

When running the init task the generated code in build.gradle isn't clearly separated from existing code.
For example:
golang {
    packagePath = 'github.com/bmuschko/link-verifier'
}dependencies {
    golang {
        build name:'github.com/mvdan/xurls', version:'d315b61cf6727664f310fa87b3197e9faf2a8513', transitive:false
        test name:'github.com/stretchr/testify', version:'69483b4bd14f5845b5a1e55bca19e954e827f1d0', transitive:false
    }
}

The task should identify if a new line is needed and clearly separate the logic so the build file stays readable.

Thank you for your advice, it will be fixed in next release.

Fixed by 0.6.5.


RuntimeError while running RAP for gsm8k dataset using exllama

Following the readme.md, I tried to run RAP for gsm8k using exllama, with the recommended instruction:
CUDA_VISIBLE_DEVICES=0,1 python examples/RAP/gsm8k/inference.py --base_lm exllama --exllama_model_dir my/path/to/Llama-2-7B-Chat-GPTQ --exllama_lora_dir None --exllama_mem_map '[16,22]' --n_action 1 --n_confidence 1 --n_iters 1 --temperature 0.0
but encountered following RuntimeError:
`Using the latest cached version of the dataset since gsm8k couldn't be found on the Hugging Face Hub
Found the latest cached dataset configuration 'main' at /home/lzy/.cache/huggingface/datasets/gsm8k/main/0.0.0/1505e1f9da07dd20 (last modified on Sun Aug  4 12:00:13 2024).
gsm8k:   0%|                                                                    | 0/1319 [00:00<?, ?it/s]
Janet’s ducks lay 16 eggs per day. She eats three for breakfast every morning and bakes muffins for her friends every day with four. She sells the remainder at the farmers' market daily for $2 per fresh duck egg. How much in dollars does she make every day at the farmers' market?
/home/lzy/Desktop/Interact/llm-reasoners-main/reasoners/lm/exllama_model.py:119: UserWarning: max_new_tokens is not set, we will use the default value: 200
warnings.warn(f"max_new_tokens is not set, we will use the default value: {self.max_new_tokens}")
/home/lzy/Desktop/Interact/llm-reasoners-main/reasoners/lm/exllama_model.py:122: UserWarning: do_sample is False while the temperature is non-positive. We will use greedy decoding for Exllama
warnings.warn(
/home/lzy/Desktop/Interact/llm-reasoners-main/reasoners/lm/exllama_model.py:144: UserWarning: the eos_token '\n' is encoded into tensor([29871,    13]) with length != 1, using 13 as the eos_token_id
warnings.warn(f'the eos_token {repr(token)} is encoded into {tokenized} with length != 1, '
MCTSAggregation: no answer retrieved.
Case #1: correct=False, output=None, answer='18';accuracy=0.000 (0/1)
gsm8k:   0%|                                                          | 1/1319 [00:09<3:35:53,  9.83s/it]
A robe takes 2 bolts of blue fiber and half that much white fiber.  How many bolts in total does it take?
gsm8k:   0%|                                                         | 1/1319 [00:34<12:35:08, 34.38s/it]
Traceback (most recent call last):
File "/home/lzy/Desktop/Interact/llm-reasoners-main/examples/RAP/gsm8k/inference.py", line 155, in
fire.Fire(main)
File "/home/lzy/anaconda3/envs/agent/lib/python3.10/site-packages/fire/core.py", line 143, in Fire
component_trace = _Fire(component, args, parsed_flag_args, context, name)
File "/home/lzy/anaconda3/envs/agent/lib/python3.10/site-packages/fire/core.py", line 477, in _Fire
component, remaining_args = _CallAndUpdateTrace(
File "/home/lzy/anaconda3/envs/agent/lib/python3.10/site-packages/fire/core.py", line 693, in _CallAndUpdateTrace
component = fn(*varargs, **kwargs)
File "/home/lzy/Desktop/Interact/llm-reasoners-main/examples/RAP/gsm8k/inference.py", line 146, in main
rap_gsm8k(base_model=base_model,
File "/home/lzy/Desktop/Interact/llm-reasoners-main/examples/RAP/gsm8k/inference.py", line 69, in rap_gsm8k
accuracy = evaluator.evaluate(reasoner, num_shot=4, resume=resume, log_dir=log_dir)
File "/home/lzy/Desktop/Interact/llm-reasoners-main/reasoners/base.py", line 232, in evaluate
algo_output = reasoner(self.input_processor(example),
File "/home/lzy/Desktop/Interact/llm-reasoners-main/reasoners/base.py", line 183, in call
return self.search_algo(self.world_model, self.search_config, **kwargs)
File "/home/lzy/Desktop/Interact/llm-reasoners-main/reasoners/algorithm/mcts.py", line 314, in call
self.search()
File "/home/lzy/Desktop/Interact/llm-reasoners-main/reasoners/algorithm/mcts.py", line 284, in search
path = self.iterate(self.root)
File "/home/lzy/Desktop/Interact/llm-reasoners-main/reasoners/algorithm/mcts.py", line 188, in iterate
self._simulate(path)
File "/home/lzy/Desktop/Interact/llm-reasoners-main/reasoners/algorithm/mcts.py", line 249, in _simulate
self._expand(node)
File "/home/lzy/Desktop/Interact/llm-reasoners-main/reasoners/algorithm/mcts.py", line 224, in _expand
node.state, aux = self.world_model.step(node.parent.state, node.action)
File "/home/lzy/Desktop/Interact/llm-reasoners-main/examples/RAP/gsm8k/world_model.py", line 96, in step
outputs = self.base_model.generate([model_input] * num,
File "/home/lzy/Desktop/Interact/llm-reasoners-main/reasoners/lm/exllama_model.py", line 163, in generate
decoded = self.generate_simple(self.generator, inputs[start:end], max_new_tokens=max_new_tokens,
File "/home/lzy/Desktop/Interact/llm-reasoners-main/reasoners/lm/exllama_model.py", line 200, in generate_simple
generator.gen_begin(ids, mask=mask)
File "/home/lzy/Desktop/Interact/llm-reasoners-main/exllama/generator.py", line 186, in gen_begin
self.model.forward(self.sequence[:, :-1], self.cache, preprocess_only = True, lora = self.lora, input_mask = mask)
File "/home/lzy/Desktop/Interact/llm-reasoners-main/exllama/model.py", line 972, in forward
r = self._forward(input_ids[:, chunk_begin : chunk_end],
File "/home/lzy/Desktop/Interact/llm-reasoners-main/exllama/model.py", line 1058, in _forward
hidden_states = decoder_layer.forward(hidden_states, cache, buffers[device], lora)
File "/home/lzy/Desktop/Interact/llm-reasoners-main/exllama/model.py", line 536, in forward
hidden_states = self.self_attn.forward(hidden_states, cache, buffer, lora)
File "/home/lzy/Desktop/Interact/llm-reasoners-main/exllama/model.py", line 440, in forward
new_keys = cache.key_states[self.index].narrow(2, past_len, q_len).narrow(0, 0, bsz)
RuntimeError: start (3072) + length (6) exceeds dimension size (3072).`

Hi,
Our prompt is designed for base models (without instruction tuning or RLHF). Using a chat model may lead to unexpected output and the program may not parse it correctly. To debug, you may print the raw outputs of the LLM.


Replace "ObjectRepo current" with "PollRepo default"

Replaces ObjectRepo current with PollRepo default to better reflect intended use. While the ObjectRepo technically can manage any Object, in our case we want to manage Polls.

Pull Request Test Coverage Report for Build 862998048

0 of 7 (0.0%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 0.0%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/Liquid-Core.package/LiquidParticipantMenu.class/instance/importPollDraftWithId..st
0
1
0.0%

packages/Liquid-Core.package/LiquidPollDraft.class/instance/startSurveyWithId..st
0
1
0.0%

packages/Liquid-Core.package/LiquidHostMenu.class/instance/savePoll.st
0
2
0.0%

packages/Liquid-Core.package/LiquidHostMenu.class/instance/downloadPoll.st
0
3
0.0%

Totals

Change from base Build 856936560:
0.0%

Covered Lines:
0

Relevant Lines:
793

💛  - Coveralls


Bug: Not unique linearGradient id when component is used multiple times

Problem:
When we use a component multiple times on a page as a form control, we get glitches when editing: the fill of the stars jumps and/or becomes the same in different components with the same value.
Reason:
the same id for linear SVG gradients on the same document page
Solution option:
Add id uniqueness, e.g. add uuid or config.name to id
<!--  Stars.svelte -->

<Star id={config.name+id} .../>

Hi @korzhanov, this is indeed an intriguing bug. After investigating, I was able to replicate the error and resolved it in this commit: https://github.com/ErnaneJ/svelte-star-rating/commit/5d0be19640724b650051f78c37aedc1410f2d40a. I'll be releasing the new version 1.1.5 shortly, which will include this fix. Thank you!


Bug: middleware: ack flood of gRPC requests

Describe the bug
"bos fees" issues a lot of gRPC requests, which need to be acked within 2 seconds. This doesn't always happen, presumably because lnd-manageJ doesn't have enough threads working on the requests.
To Reproduce
Run bos fees --set-fee-rate 123 --to x a few times.
Expected behavior
lnd-manageJ ACKs all gRPC requests in time.

I don't know how to fix this.
Workaround: increase the delay in `.lnd/lnd.conf:
[rpcmiddleware]
rpcmiddleware.intercepttimeout=10s


TS2346: Supplied parameters do not match any signature of call target.

Note: for support questions, please use one of these channels: Chat: AngularClass.slack or Twitter: @AngularClass
Hi,
I've tried to access the angularclass.com support links to make sure before i post this one if this is a bug or not but with no success.
I wonder why when i clone this repository and do npm intall and then when i try to inject any simple service
into the home or about component I get this error
"
[at-loader] Checking finished with 1 errors
[at-loader] compiled\src\app\home\home.component.ngfactory.ts:36:20
TS2346: Supplied parameters do not match any signature of call target.
"
here is my test.service.ts file
"
import { Injectable } from '@angular/core';
@Injectable()
export class Test {
constructor() {
}

}
"
this is the home.component.ts modification i make
import { Test } from '../services/test.service';
providers: [
AppState,
Title,
Test
],
// TypeScript public modifiers
constructor(
public appState: AppState,
public title: Title,
private test:Test
) {}
"
and simply i cannot get away from the error I'm getting.
when i take a look at the compiled\src\app\home\home.component.ngfactory.ts:36:20 file
i see this :
"
constructor(p0:any,p1:any) {
this._changed = false;
this.context = new import0.HomeComponent(p0,p1);
}
"
So obviously the constructor parameter signature is not right and I cannot find the spot where I can fix this and I wonder if this is some sort of a bug or something else ?

Have the same issue. I found it related to @angularclass/hmr package and custom bootstrapping app has because of HMR. Looks as it require public appState: AppState to be passed there always. In my project I'm using @ngrx/store and not require HMR in a way AngularClass have it by default, so removed.


消除编译告警

消除编译告警

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Extra casting not necessary

Hello,
Actual:
interface ITest{test:string}; let test:ITest=<ITest>{}
Expected
interface ITest{test:string}; let test:ITest={}
I think {} should be considered as ITest by default instead of any.
Thanks in advance for your feedback.

interface ITest{test?:string}; let test:ITest={}

Sorry hit comment too soon.
ITest requires one member test. the empty object does not have that. they are not assignable.
you can make test optional in ITest:
interface ITest{test?:string}; let test:ITest={}


added pubsub options to bus, added bus code for pubsub

added pubsub options to bus, added bus code for pubsub
this should not break any existing functionality
initial PR, this will need refinement  and unit tests

We should add go modules as part of this commit as well.


Quests are deducting stamina when they aren't meant to

Certain story quests eg 'The Moovers and the Shakers' display a stamina cost of 0 but actually deduct 4 stamina, leading to an error when attempting to complete the dungeon

Closed by #439


add headless about pages endpoints

Describe the problem
I want to access to the content of the about pages without having to scrape the page.
Describe the proposed solution
Add the following api endpoints:

/api/about/it.json
/api/about/en.json

We want the output json to have shape  PostInfoDto.
Alternatives considered
None
Importance
nice to have

@FaberVitale please assign it to me


[Proposal] Make possible to download file from GitHub private repository

Recently I needed to download a file from a private GitHub repo, in particular the zip archive from release's assets. But the plugin didn't allow me to do that, because it didn't know the valid token to access the private release.
What do you think about adding an ability to specify an authorization credentials to download the file?

@maxoly I've created a custom action to download release's asset and I think I can make it as an another plugin. Do you mind if I use most of the code from your plugin?


Simplex noise in procgen lib

This PR inlines the simplex lib into the useProcGen metaversefile api.
Simplex noise is used all over the procedural generation apps. They all import their own copies of this library, which wastes memory and makes it harder to write new procedural apps since you need to copy boilerplate files.

Also includes the MultiSimplex wrapper for octaved simplex noise.


Improve: custom primary key and custom mapping

Improve support for syncing an entity with custom mapping and custom primary key.

:star:


LookDev transfer Error

Example move
Process to get to produce the bug:

Create a model for assetA.
Create a lookDev for assetA.
Create model assetB.
Create lookDev for assetB.
Load ModelA and Lookdev into a new scene
Load ModelB and Lookdev into same scene
Load Lookdev for ModelA again and it will get the ModelB lookDev and it can't be changed back with the loader.

Seems like its always the First models and lookDev that you load that transfers the shader to the wrong asset. As can be seen in the video even though I create a completely new scene the bug is still there.

Thanks @CalleHalldin, having a look at this tomorrow morning. Make sure the steps to reproduce does reproduce the problem, that way it'll be much quicker to fix.

Sorry I haven't gotten into this yet.
@LegacyID1991, you might be able to dive into this. The problem must be somewhere in the loader for this family, see here.
I would start printing things, and see if you can pinpoint where in the loader the problem occurs. Likely in the maya.apply_shaders function. If you print the relationships variable, it'll give you a dictionary. You can use that dictionary to manually call apply_shaders and debug from there.

@LegacyID1991 Did you have a look at this? If not, let me know here so I know whether I should look at it.

I am sorry to say that I have been so slammed with other work duties that I haven't gotten to this one yet.

That's ok, I just need to know whether I should look into it so we don't work on it at the same time. Would you like me to take over?

go right ahead.
😄

Ok, I've found the problem.
The problem is that it assigns to meshes based on the name of the shader group, but if the shader group has the same name in both look dev publishes, such as blinn1SG, the it gets confused and assigns the wrong one.
Putting together a fix now.

Fixed, @LegacyID1991 is updating the pipeline shortly.


Conventions around authnz

For any kind of pluggable authentication layer (implemented through a middleware interceptor), do we want principal details to show up under a predictable, conventional name (or names) attached to the request? (Eg, USER, PRINCIPAL, AUTH, FN_AUTH, ...?)
For authorization decisions, the potential scope of how these are specified (in a function* config) will depend upon the implementation - however, I wonder if there's still room to say, "find your authz policy attached to the FOO configuration parameter".
As a user, it'd be nice if my functions could depend upon a consistent approach for this - even if the actual details of the authnz configuration depended upon the platform that I was deploying to.

"function" here meaning app, route, namespace, what-have-you.

I’m closing this as I think we are covering it elsewhere


Missing pimd.conf and pimd never sets up any Rendez-vous Point(s)

@ocochard reports in issue #57:

With an "empty" on unexisting pimd.conf, the BSR address was always <IP_ADDRESS> and
it never add a rendez-vous point, then never update its mrouting table.

It's not a blocking problem, the RC script in charge of starting this deamon can check the presence of the pimd.conf.
Providing more information about the "unexistant configuration file" behavior:
[root@pim-router]~# rm /usr/local/etc/pimd.conf
[root@pim-router]~# pimd -d
debug level 0xffffffff (dvmrp_detail,dvmrp_prunes,dvmrp_routes,dvmrp_neighbors,dvmrp_timers,igmp_proto,igmp_timers,igmp_members,trace,timeout,packets,interfaces,kernel,cache,rsrr,pim_detail,pim_hello,pim_register,pim_join_prune,pim_bootstrap,pim_asserts,pim_cand_rp,pim_routes,pim_timers,pim_rpf)
06:29:08.205 pimd version 2.3.2-beta1 starting ...
06:29:08.205 Got 262144 byte send buffer size in 0 iterations
06:29:08.205 Got 262144 byte recv buffer size in 0 iterations
06:29:08.205 Got 262144 byte send buffer size in 0 iterations
06:29:08.205 Got 262144 byte recv buffer size in 0 iterations
06:29:08.205 Getting vifs from kernel
06:29:08.205 Installing em0 (<IP_ADDRESS> on subnet 10.0.12/24) as vif #0 - rate 0
06:29:08.205 Installing em1 (<IP_ADDRESS> on subnet 10.0.23/24) as vif #1 - rate 0
06:29:08.205 Getting vifs from /usr/local/etc/pimd.conf

=> No warning message about a "not found" file
06:29:08.206 Interface em0 comes up; vif #0 now in service
06:29:08.207 query_groups(): Sending IGMP v3 query on em0
06:29:08.207 Send IGMP Membership Query     from <IP_ADDRESS> to <IP_ADDRESS>
06:29:08.207 SENT    36 bytes IGMP Membership Query     from <IP_ADDRESS>       to <IP_ADDRESS>
06:29:08.207 SENT    46 bytes PIM v2 Hello              from <IP_ADDRESS>       to <IP_ADDRESS>
06:29:08.207 Interface em1 comes up; vif #1 now in service
06:29:08.208 query_groups(): Sending IGMP v3 query on em1
06:29:08.208 Send IGMP Membership Query     from <IP_ADDRESS> to <IP_ADDRESS>
06:29:08.208 SENT    36 bytes IGMP Membership Query     from <IP_ADDRESS>       to <IP_ADDRESS>
06:29:08.208 SENT    46 bytes PIM v2 Hello              from <IP_ADDRESS>       to <IP_ADDRESS>
06:29:08.208 Interface register_vif0 comes up; vif #2 now in service

=> With no configuration file it enable PIM on all interfaces (em0 and em1 here): great, this mean without configuration file it should be working.
(...)
Virtual Interface Table ======================================================
Vif  Local Address    Subnet              Thresh  Flags      Neighbors
---  ---------------  ------------------  ------  ---------  -----------------
  0  <IP_ADDRESS>        10.0.12/24               1  DR NO-NBR
  1  <IP_ADDRESS>        10.0.23/24               1  DR NO-NBR
  2  <IP_ADDRESS>        register_vif0            1

 Vif  SSM Group        Sources

Multicast Routing Table ======================================================
--------------------------------- (*,*,G) ------------------------------------
Number of Groups: 0
Number of Cache MIRRORs: 0
------------------------------------------------------------------------------

Candidate Rendezvous-Point Set ===============================================
RP address       Incoming  Group Prefix        Priority  Holdtime
---------------  --------  ------------------  --------  ---------------------
------------------------------------------------------------------------------
Current BSR address: <IP_ADDRESS>

=> But there is nothing configured as Rendezvous-point (no canditate rp, no BSR)… then the "default" empty configuration prevent to route mcast.
If we start a mcsat sender at one interface (em0 here), pimd didn't update its mcast routing table:
06:46:38.189 Cache miss, src <IP_ADDRESS>, dst <IP_ADDRESS>, iif 0
06:46:38.190 create group entry, group <IP_ADDRESS>
06:46:39.692 Cache miss, src <IP_ADDRESS>, dst <IP_ADDRESS>, iif 0
06:46:39.693 create group entry, group <IP_ADDRESS>
06:46:41.183 Cache miss, src <IP_ADDRESS>, dst <IP_ADDRESS>, iif 0
06:46:41.184 create group entry, group <IP_ADDRESS>

Virtual Interface Table ======================================================
Vif  Local Address    Subnet              Thresh  Flags      Neighbors
---  ---------------  ------------------  ------  ---------  -----------------
  0  <IP_ADDRESS>        10.0.12/24               1  DR NO-NBR
  1  <IP_ADDRESS>        10.0.23/24               1  DR NO-NBR
  2  <IP_ADDRESS>        register_vif0            1

 Vif  SSM Group        Sources

Multicast Routing Table ======================================================
--------------------------------- (*,*,G) ------------------------------------
Number of Groups: 0
Number of Cache MIRRORs: 0
------------------------------------------------------------------------------

Candidate Rendezvous-Point Set ===============================================
RP address       Incoming  Group Prefix        Priority  Holdtime
---------------  --------  ------------------  --------  ---------------------
------------------------------------------------------------------------------
Current BSR address: <IP_ADDRESS>

06:46:42.686 Cache miss, src <IP_ADDRESS>, dst <IP_ADDRESS>, iif 0
06:46:42.687 create group entry, group <IP_ADDRESS>

And the operating system mcast routing table didn't know about the correct In-Vif/Out-vif to use:
[root@pim-router]~# netstat -g

IPv4 Virtual Interface Table
 Vif   Thresh   Local-Address   Remote-Address    Pkts-In   Pkts-Out
  0         1   <IP_ADDRESS>                               0          0
  1         1   <IP_ADDRESS>                               0          0
  2         1   <IP_ADDRESS>                               0          0

IPv4 Multicast Forwarding Table
 Origin          Group             Packets In-Vif  Out-Vifs:Ttls
 <IP_ADDRESS>       <IP_ADDRESS>               0  65535

IPv6 Multicast Interface Table is empty

IPv6 Multicast Forwarding Table is empty

=> here the In-Vif should be set to number 0 and out vif to 1.

Thank you for the thorough follow-up! This will be very useful tracking this buglet down :)
… and if course it should be possible to run pimd without a .conf file! :)

@ocochard I've looked into this a bit further now.  It seems pimd has actually been designed to not act as an RP or BSR candidate by default.  This is also in line with what Cisco does by default.
http://www.cisco.com/c/en/us/td/docs/ios/12_2/ipmulti/command/reference/fiprmc_r/1rfmult2.html
However, since we both want pimd to work out-of-the-box when there is no config file I have a proposal:

Keep things as-is, no RP/BSR candidate if settings are missing from config file (as today)
If no config file is found, warn to log but set up sane defaults, i.e. as RP and BSR candidate

Commit c99c7f9, part of v2.3.2-rc2, should work a lot better with no config file, without breaking Cisco compat defaults.

Nope, does not work.  Tested with a very simple setup:
Sender -----> R1 <----> R2 <----> R3 ------> Receiver
               ^         ^         ^
                \         \         `-------------------------- OpenBSD
                 \         `----------------------------------- Linux
                  `-------------------------------------------- Linux

No luck. Must look into this again before release.

False alarm!  After much trouble shooting the problem I had (apart from forwarding taking aaaages to start) was IP address collision in my virtual multicast test bed :smile: :+1:


Showcase Plot doesn't have link to the code

The example page at http://uber.github.io/react-vis/examples/showcases/plots doesn't show a link to the implementation. Since this is the only chart with legends enabled, this is kinda bad.
Solution: add a link to https://github.com/uber/react-vis/blob/master/packages/showcase/plot/complex-chart.js at
https://github.com/uber/react-vis/blob/2ac4544b9ea3abf0861d2a0268f2746301cefdbf/packages/showcase/showcase-sections/plots-showcase.js#L256
The code would probably look like this:
<a {...linkProps} href={exampleLink}>
  {' '}
  View Code
</a>

#1379


[NETBEANS-138] Fixed InjectedTasksSupportTest test.

Fixed InjectedTasksSupportTest test in parsing.indexing module.
The order that the indexers are run is different that the test expect.
It closes this issue NETBEANS-138 and old bug Bugzilla: 237764.

@blackleg as noone seems to know what this test should test I suggest to remove it. We gain nothing by doing a fix that can't be verified. In other cases I voted to keep the test, as the test itself was fine, but only failed because files were not donated. In this case the test itself is the problem. Are you willing to do that?

@matthiasblaesing It seems right, I will update pull request to remove the test.

@matthiasblaesing I removed the test.


Charades json

Can you share an example of the charades json file used for the train file or an example of how to format another dataset in the same manner?

The json file I used is available here:
https://github.com/piergiaj/super-events-cvpr18/blob/master/data/charades.json
There are other datasets there as well (in https://github.com/piergiaj/super-events-cvpr18/blob/master/data)
They follow a simple format:
{"Video Name/ID": {"subset": "training/testing", "duration": 23.21, "actions": [[class_id, start_seconds, end_seconds], ...]}}


App Build Automation

Automating the app-package build
Acceptance Criteria:

Ability for a user to trigger an app-package build (authentication?)
app-package is built from project provided source code in an open repository
app-package is stored in docker registry
app-package is registered in Dockstore
CWL is available through dockstore

@mcduffie please create and/or link the work tickets for the acceptance criteria noted above.

@mcduffie I noticed that the one work ticket (https://github.com/unity-sds/unity-app-build-trigger/issues/1) associated to this task is complete.  Is that the only ticket required to close this one out?  If not, can you please add any other related work tickets under the "Work Tickets" section here?

Seems like we still need to complete https://github.com/unity-sds/unity-app-build-trigger/issues/2 before closing out this ticket, if so then I will add that ticket to the "Work Tickets" section here.

@edwinsarkissian - we'll need to clean up the EC2 installs for the gitlab runners at somepoint. there is absolutely no version control happening and that's going to be an issue when trying to support dev/test/prod deployments - as what's installed on them will be a factor of when they were installed.

see https://github.com/unity-sds/unity-ads-deployment/blob/main/ci_cd/install_group_runner_x86_64_docker.tftpl#L171-L174 for examples fo what i'm talking about.

Yes, I agree. I will need to deal with version control.  However, for now the problem is that registering image with the local image registry fails without any error message, and I have no idea what is causing it. I would have expected to see some error message or warning coming out of app-pack-gen or unity-app-gen. Unfortunately, I won't be able to spend more than 5% of my time on Unity, which I have already done.  I have to come back to this early next week.


INFO:root:Waiting until the server is available.

Hi, I'm trying to use this library for a project where I would need corefence resolution. I was testing out the test.py file and this kept on showing indefinitely, how do I fix this?
Thanks in advance.

Hi, have you figured it out? I have also meet this issue.

Hi, no, I just started using something else for coreference resolution.

I had the same issue, try this stripped down version of corenlp.py I made, it also fixed the issues I had for mac (python didn't have the permission to view the current connections so it couldn't verify which ports were being used so instead I used one port and run corenlp through a with: to make it it is closed properly).
This version is a little bit of a botch, but hey, it works.
https://github.com/matanrak/NaturalIntelligenceExercise/blob/master/corenlp.py

The solution for me was to add '--add-modules java.se.ee' after the 'java' when opening the server, this fixes compatibility errors with java 9.
Also you might want to take a look at a stripped down version of corenlp.py I made:
https://github.com/matanrak/NaturalIntelligenceExercise/blob/master/corenlp.py

Okay, thanks!

my issue was with 32b java installed, try 'java -d64' in cmd and see if it's working! if it isn't just install it and run it again.

Hi! antoine,have you solved this problem? @antoineChammas

兄弟这个问题解决了吗？：） @xiedaxia1hao

@kevinmraz Not yet.. I just used the other package..

fine,thx! btw, could u please tell me what another package u used?

@kevinmrazh Hi! If you are doing Chinese NLP, you can use jieba

Of course I have already used it. But to b honest, it did not work very well~~

I have the same issue. I try to increase the sleeping time under CoreNLP.py line 114 and it works.

I have the same issue. I try to increase the sleeping time under CoreNLP.py line 114 and it works.

Thanks. I had the same issue and increasing it to 10 made it work.


A Linux based dokcer image with all the required libraries

Dear All,
I am building a docker image which includes python 2.7 , jupyter, dask, fastparquet, llvm and all the relevant dependencies. It is available here:
https://github.com/QuantScientist/deep-ml-meetups/blob/master/nice-docker/README.md
https://hub.docker.com/r/quantscientist/deep-ml-meetups/builds/
It is my hope that it would be useful to others,
Let me know if you have any requests, parquet-cpp is on my next to do list :)

Note that I do not know how complicated building parquet-cpp is, but you might be interested in fastparquet, which does have python requirements (cython, numba, thrift), but conda-installs without issues.

fastparquet is already installed on the docker image.

ok ;)

Thanks for informing us @QuantScientist .  I'm going to close this down for now.  Please feel free to reopen if further conversation is warranted.


suppress the alert

suppress this error. by design.
https://lgtm.com/projects/g/microsoft/azuredatastudio/snapshot/51fc833f0f88a1758dcfc2ec42b3c4da5631d69e/files/extensions/resource-deployment/src/services/tools/azCliTool.ts?sort=name&dir=ASC&mode=heatmap#L63

Ignore these failures, they're here because your branch isn't up to date with master.


Removing a platform should remove the corresponding entry from config.xml

:+1:

I see a check for whether the user has done "taco platform add platform@version", but that check will also apply to "taco platform rm platform@version". Cordova seems to basically do nothing if you specify such an argument, do we behave appropriately?

"cordova platform remove platform@version" is a no-op. TACO should and does not update the platforms or config.xml in this case.

Sounds good. Looks good to me.


windows scripts for bootstrap and destroy

I ported the bootstrap and destroy scripts to Windows for use with CRC.
The scripts match the original scripts and were tested on:
CodeReady Containers version: 1.35.0+751824a9
OpenShift version: 4.9.5 (bundle installed at C:\Program Files\CodeReady Containers\crc_hyperv_4.9.5.crcbundle)

@jduimovich I'm hesitant to include this, as it create an expectation that this is fully supported and would require us to maintain two sets of scripts (unless you are willing to add your name to the file as the maintainer and to keep them up to date yourself).
But also, can't Windows users can just use the existing scripts with WSL?

I'd say, it would be good to have @jduimovich as the maintainer of the windows scripts. 🚨

But also, can't Windows users can just use the existing scripts with WSL?

I tried WSL and I was getting errors with weird paths and filenames mixing G:\path and /mnt/g/path but perhaps we could fix by carefully writing the scripts. I can investigate because having two scripts is a pain. It may be that windows and linux binaries are being mixed as you can directly run kubectl.exe from shell.    The other issue is cr/lf messes in WSL, if you use windows / vscode and run in WSL you get a bunch or "\r" errors and the scripts fail.
If I can get WSL to work we can fix.

@jduimovich My guess is if the Linux x64 version of the kubectl binary is installed in WSL, and that is what is first on your PATH,  that should resolve the issues you are seeing re: supported path formats.
And yes, CR/LF issues with Git were always terrible when I was on Windows, and autocrlf never seemed to do what I wanted in all cases, so I fully sympathize :upside_down_face:. If you git clone from inside WSL that should give you the line endings you need, but there's no guarantee that editing the text from within Windows won't mess that up again.

I tested it and the kubectl was fine after many tries to find which one was
running :)
Turns out the kustomize prints an error about checking symlinks which come
up as "g:\mnt" and messes from there.
I deleted all my binaries and reloaded and it seems to work so we may be
able to get away with WSL only.
The CR/LF really only seems to hurt the scripts for now, and yes, checkout
in windows breaks WSL, checkout in WSL and is ok.
The issue is depending on where you check it out, it may show all (24+)
files changed if you look from the "other side"
The simple fix is rm, and checkout again and it doesn't show modified :) If
you dos2unix the files they also claim to be changed ala cr/lf differences
I think I'm ok without the windows scripts for now.
I have a new script which I'm experimenting with local dev which redirects
the gitops repositories via a patch and a  kustomize directory overlay. It
means that we can leave the originals that target sandbox in place to
minimize the risk that someone sends in changes which includes references
to their private repo when testing components.
On Thu, Nov 18, 2021 at 8:21 AM Jonathan West @.***>
wrote:

@jduimovich https://github.com/jduimovich My guess is if the Linux x64
version of the kubectl binary is installed in WSL, and that is what is
first on your PATH, that should resolve the issues you are seeing re:
supported path formats.
And yes, CR/LF issues with Git were always terrible when I was on Windows,
and autocrlf never seemed to do what I wanted in all cases, so I fully
sympathize 🙃. If you git clone from inside WSL that should give you the
line endings you need, but there's no guarantee that editing the text from
within Windows won't mess that up again.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/redhat-appstudio/infra-deployments/pull/11#issuecomment-972860084,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB33CXXJG6HPAAJA2DJEYJ3UMT4UXANCNFSM5IF7R5CQ
.
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.

I have verified WSL is proven to work (with messy cr/lf and other "nits") but the scripts work.  The solution was to ensure recent versions of kubectl, kustomize. some random version of kustomize had issues with symlinks for some reason. I deleted all and found fresh, reinstalled and scripts started to work.  I am closing without merge


change shrink item animation to be on the image container and not on the image content

fix - https://wix.monday.com/boards/4776391986/pulses/5238748140?filter=XQAAAAK6AAAAAAAAAABBqQqHk62Wwa8yt9cbbXI48HpL2ei1XixEiNVOhiWmf4IrTn4lrcVxz0d8EzlqDpxL1dc7i9JjI1nNW0HEFgqaXhsDsHWTHjMEE6-qlUKWgWTaunFWUdBXUlnD6ksBNUF-ix8iV3_U-Feu4Xakffa5TWjTljysumsFSTUwEWM70MyKZaOO_hr7RKIeSDv_FebgsiEliuxt-ZRFyK4wE___kOAAAA
before

after

@liatv and the team. This is an Open source, please try to elaborate what the fixes are w/o adding internal Wix links. (or anything Wix)


ARM Build

Hello,
I'm deploying to itch from my raspberry pi. So far, I compiled my own version of butler for this, but I'm getting errors like Version check failed: HTTP 404: https://broth.itch.ovh/butler/linux-arm-head/LATEST, and no being a Go person myself, it was quite an adventurous process to get there. It would be great to just get a compiled version from broth.

I don't really have access to any ARM machines, so there's no official builds.
I know the DragonRuby folks are doing ARM builds of butler though!

BTW the version check error should just be a warning right? It doesn't actually prevent butler from working?

Yes, it seems to just be a warning. How are you building the other versions?
I think Travis CI supports ARM.

We also work from solar/rpi, I would love an ARM build <3

I don't work at itch anymore — I think I remember others making butler builds for arm64, so it should be possible!

Ah! Understood thanks for the head's up :) I'll try to get the golang toolchain going on the Pinebook.

Ah! Understood thanks for the head's up :) I'll try to get the golang toolchain going on the Pinebook.

Hey there - any progress on this? I'd also love an armv7 build of butler so I can use my raspberry pi 4 as a build machine!

@aidanoflann Did you try go build after checking out the repo. As far as I know the entire project should build on arm but I have not personally tested it

@aidanoflann Did you try go build after checking out the repo. As far as I know the entire project should build on arm but I have not personally tested it

Oh thanks! That was actually much more straightforward than I thought. As you suggested I just ran go build from the root dir of the repo.


Made accordion heading tab-able for IE9-10

Having an empty href tag works with modern browsers but not with IE9 or IE10. Adding tabindex="0" makes the accordion heading tab-able without mixing up the tab order.

Yo!
This is a small patch, shouldn't be troublesome to merge, not a priority right now (we need 1.3 right now).
@wesleycho Do you have experience with this?

Landed as 6abad509cd4d44c3ca432f2f21c9ecea0a206b53


[Bot] Update API inference documentation

This PR automatically regenerates the API inference documentation by running:
cd scripts/api-inference
pnpm run generate

This PR was automatically created by the Update API Inference Documentation workflow.
Please review the changes before merging.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

(PR autogenerated from https://github.com/huggingface/hub-docs/pull/1480, let's wait before merging)


Add contributing.md file

it'd be nice to have some guidelines so new users know:

what is expected from contributors;
how to report issues;
tips on design/cleanup proposals;
how to talk to the core team/other users;
the language conventions;
how to use commit messages;
how does PR review works;


Dependabot broken since ~March 13th

It looks like dependabot hasn't run since March, so the status pages are mostly stale as they are stuck on the versions as of then.
If this is intentional, please consider another way to update submodule'ed tools automatically.  Thanks.

Thank you for the report! I've refreshed the dependabot settings and it seems to have helped, new PRs are being created right now


Update onhand-cleanup.md

More precise possible user impact

:x: Organization membership check failed
Members of the MicrosoftDocs organization should use the private repo. Please move your changes to https://github.com/MicrosoftDocs/Dynamics-365-Operations, and close this pull request.

:x: Organization membership check failed
Members of the MicrosoftDocs organization should use the private repo. Please move your changes to https://github.com/MicrosoftDocs/Dynamics-365-Operations, and close this pull request.


Add GitHub actions (auto assign and label)

Issue #70 solved
Type of PR

[ ] Bug fix
[X] Feature enhancement
[ ] Documentation update

Description
This pull request will run GitHub actions, in order to:

Assign the issues to the author
Label the issues with hacktoberfest tags

Explanation
Before:

New issues were sadly empty

After:

New issues immediately get labels and are assigned

Checklist

[X] I have performed a self-review of my code.
[X] I have tested the changes thoroughly before submitting this pull request.
[X] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[X] I have commented my code, particularly in hard-to-understand areas.

@JCarlosR can you also  make it  for prs

@AnitSarkar123 I just added screenshots to the PR description. Sry for so many commits, I just wanted to make it was working properly on my fork. Feel free to squash when merging please.


Fix yarn install user-agent check

Fix #285

already committed these changes here #282 should I revert?

@AyoubBenAl1 yes.

Closing this because i've included it and updated tests in #276


THANKS! Found a couple bugs? Should report?

Hey Nicolás, thanks for the library, very handy, and much smaller than the alternatives. Thank you!
I found a couple bugs, would you like me to report them? I don't want to report if you don't care :)
Cheers!
D

Sure


Request for References / documentation of tilemaps

I'm trying to understand how rpgMakerMV works/ renders, and come across this lib. And while the code is not too long to read through, I believe that there should still be an documentation if at least what functions/methods does what by looking up a reference.
I'm sorry if there's already a references/ documentations of this tilemap plugin/lib, but I could not find it. Note that I'm not saying the pixi reference, but the pixi-tilemap reference.

The lib was made for RMMV and then re-used in many projects. Its a low-level algo, its not tied to any of editors or formats. The "window around camera" optimization exists only in rpgmaker demo, in all other cases you have to make your own high-level algo that is using the plugin.
Unfortunately, I do not have time, and I will not have time to document it. I leave those kind of things to community, especially when there are many people who make PR's for my plugins.


MM-14872 - Updating autocomplete positioning

Summary
MM-14872 - Updaitng autocomplete positioning
Ticket Link
https://mattermost.atlassian.net/browse/MM-14872

Hey @asaadmahmood, I don't feel knowledgeable enough to review this code area. Pleas ask someone else for a review.

@hanzei Makes sense.

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: https://i-0a811afe5b357f3fa.test.spinmint.com
Test Admin Account: Username: sysadmin | Password: sysadmin
Test User Account: Username: user-1 | Password: user-1
Internal IP: <IP_ADDRESS>
Instance ID: i-0a811afe5b357f3fa

Spinmint test server destroyed

Can somebody help merge + cherry pick?


[wip] Changing affiliation from code to name

This aims to deprecate https://github.com/pulibrary/tigerdata-app/pull/937

coverage: 84.288%. remained the same
when pulling f6dd0e5390a4ae2ae323559bcacc675d06d8b0b1 on i931-jrgriffiniii-affiliation
into a04090ac2cc28150b94cfff748cb3a45724dfd25 on main.

This now has tests passing, but the Mediaflux version local to my development environment triggered the failure of certain tests. I am re-enabling these and shall clean my commits.


Sanjo/vault kex

@SanjoDeundiak This got merged but Github is confused. It's top of develop at the moment.


[Feature Request] "Record Based FlowField" or "Expression Filled Variable"

To reduce the complexity of NAV you should implement a fuctionality i would name "Record Based FlowField" or "Expression Filled Variable".
There are many redundant variables in NAV- records like "Qty. Shipped not Invoiced" or "Outstanding Quantity" in T37. These variables have no real new information because they have a fixed calculation of other variables like  "Qty. Invoiced" -"Qty. Shipped" respectively "Quantity" -"Qty. Shipped".
Handling like a automaticaly calcuated flowfield would reduce the effort in code to update these fields, and would reduce the need of  knowledge about "to validate or not to validate" or to call functions to upate these variables.
Another way could be to assign these varibales by an expression- property like 'Value="Qty. Invoiced" -"Qty. Shipped"' which is automaticaly triggered by assigns (with validate or not) to the used Variables. This would have the advantage not to break other NAV- functionalites, and there would be only a small impact to the NAV- Structure.

Thanks for the suggestion. We will consider it for the long term backlog.

Hi, thank you for your suggestion! We are not prioritizing this for the immediate future, but if still relevant for BC we recommend you submit this at aka.ms/BCIdeas, or vote up the idea if its already there. We're constantly monitoring top Ideas and will consider them for a future release.


chore: add a failed test for .toHaveProperty('')

Summary

When I try using .toHaveProperty() with an empty string:
expect({ '': 1 }).toHaveProperty('', 1);

It gets the following error:
    expect(received).toHaveProperty(path, value)

    Expected path: ""

    Expected value: 1
    Received value: {"": 1}

I believe it's a bug.
Test plan

Added a failed test.

This regression was introduced from v27.4.0. I can confirm it works in v27.3.0

Added a quick fix for the regression. @SimenB

Codecov Report

Merging #12251 (9317925) into main (5d483ce) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main   #12251   +/-   ##
=======================================
  Coverage   67.51%   67.51%
=======================================
  Files         328      328
  Lines       17246    17249    +3
  Branches     5071     5072    +1
=======================================
+ Hits        11643    11646    +3
  Misses       5570     5570
  Partials       33       33

Impacted Files
Coverage Δ

packages/expect/src/utils.ts
96.09% <100.00%> (+0.05%)
:arrow_up:

Continue to review full report at Codecov.

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


Build V2.20.0 and skip osx arm64

Checklist

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

Replaces #158 to build ray2.20.0. Since osx arm64 cross platform builds are not working, remove it from conda-forge.yml. Rerendering pulled in python3.12, but #157 is not ready, so add a skip for python>311

CI is passing. Any reason not to merge this?

Please hold
On Wed, May 22, 2024, 07:08 Matti Picus @.***> wrote:

CI is passing. Any reason not to merge this?
—
Reply to this email directly, view it on GitHub
https://github.com/conda-forge/ray-packages-feedstock/pull/161#issuecomment-2124632780,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAPRFEO3IG6PZGW6OSH4S3ZDSDCPAVCNFSM6AAAAABIC4X4C6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMRUGYZTENZYGA
.
You are receiving this because your review was requested.Message ID:
@.***>


Update array check

instanceof can be misleading at times. Specifically, when dealing with
arrays created in different contexts, the array Prototype will be
different allowing an array to be returned from .get calls.
This change uses Array.isArray as suggested by #566.
Closes #566

Duplicate of https://github.com/libxmljs/libxmljs/pull/462
As this repository is mainly dead I forked it a while ago. This is fixed in my fork (as well as several other things)
https://github.com/marudor/libxmljs2


Azure Communication Services Phone Numbers Browse Functionality

Description
Azure Communication Services Phone Numbers Browse Functionality
(Work in Progress)

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-communication-phonenumbers


Blazor scoped CSS does not hot reload

Steps to reproduce:

Close the repo
Add the following to photino.Blazor\Samples\Photino.Blazor.Sample\Pages\Counter.razor.css:

h1 {
   color: yellow;
}

Run the app with hot reload (Rider F5, dotnet watch run, VS F5)
Change color: yellow to color: red; and save

After saving, nothing happens, you need to hit F5 in the app to get it to reload the CSS.
Btw, on Visual Studio 2022 Version 17.3.0 Preview 2.0 the scoped CSS doesn't work at all :( dotnet watch or Rider work with F5

With Photino there is a native window running with a browser control. Unless you're debugging Photino.Native (the C++ application), that part is not connected to a debugger. Your .NET code can be debugged, but not the native window or the browser control running in it.


Crypto::Bcrypt PASSWORD_RANGE hard-coded to an input-length-max of 51

I'm porting an app from Meteor, and wish to use the same user login data:
an email address, and a password. Meteor does this silly SHA256 hash of the plain-text password on the client before sending it to the server, where it uses node bcrypt (native or js) to process this 64bit digest with 10 rounds of bcrypt.
The issue is that node bcrypt (and possibly others?) accepts a password input length of up to 71 (72?) per the blowfish algorithm internals of bcrypt
while Crystal native Crypto::Bcrypt has a PASSWORD_RANGE of 1 to 51, which may be fine for a normal password, but not a 64bit pre-hashed password.
Is there a particular reason to limit it to 51?
https://github.com/crystal-lang/crystal/blob/3c712288ba8f8138cb734e673cc622658300c229/src/crypto/bcrypt.cr

See this anwswer on stack exchange.

Looking at the history, I introduced this limitation. I probably copied it from some place (but which one?). Looking at the discussion on stack exchange, it appears the theoretic limit is 56 bytes (not 51) and as it happens to be, it would seem the actual limit would be 72 bytes (including the leading null byte).

I'm not 100% about Crypto::Blowfish, but (most?) blowfish implementations take up to 72.
https://github.com/B-Con/crypto-algorithms/blob/master/blowfish.c
In terms of Crypto::Bcrypt
I only see the PASSWORD_RANGE limitation on the initializer Error instantiation.

awesome. thank you


Failed to Find a Submodule during Verilog Synthesis

Version
Yosys 0.37+1 (git sha1 e1f4c5c9cbb, clang  -fPIC -Os)
On which OS did this happen?
Linux
Reproduction Steps
Hi,
I am having problems with yosys synthesis. After yosys ran, I received the following error report: Module \DotProduct_block10' referenced in module \Subsystem' in cell `\u_cfblk36_inst' is not part of the design
Synthesis processes as follows:
read_verilog Subsystem_org.v
synth
write_verilog Subsystem_yosys.v
The same design file 'Subsystem_org.v' can be successfully synthesized without error in the vivado synthesis tool, proving that a yosys problem appears to have occurred.
I uploaded all the source files in the attachment, as well as the incentive file 'Subsystem_tb.v'. We hope to help you to repeat the problem as soon as possible. Look forward to hearing from you
yosys_3.zip
Expected Behavior
synthesis success
Actual Behavior
synthesis fail

As the error says; \DotProduct_block10 is not in the Subsystem_org.v file.  You need to read all of the source files before calling synth, not just the top module.  You should also specify the top level module with synth -top Subsystem.
read_verilog *.v
synth -top Subsystem
write_verilog Subsystem_yosys.v

works without error.


Celery updates

Description:

Fixes #101
The tests have gotten an overhaul, modularizing the pytest fixtures further, while collecting all strategy tests (since they were extremely similar) in a single file, supporting running through all backends as well.
The general backend split has been tightened as well, moving parts "upstream" (e.g., from a specific strategy to the base strategy class or from the base strategy class to the backend client class, etc.)
Implement Celery-specific support for testing the celery/remote strategy and the updates it has received in OTEAPI Core.
Type of change:

[x] Bug fix.
[ ] New feature.
[ ] Documentation update.

Checklist for the reviewer:

This checklist should be used as a help for the reviewer.

[ ] Is the change limited to one issue?
[ ] Does this PR close the issue?
[ ] Is the code easy to read and understand?
[ ] Do all new feature have an accompanying new test?
[ ] Has the documentation been updated as necessary?

Codecov Report
Patch coverage: 86.77% and project coverage change: -2.90 :warning:

Comparison is base (58de422) 93.40% compared to head (0a70c52) 90.51%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #103      +/-   ##
==========================================
- Coverage   93.40%   90.51%   -2.90%
==========================================
  Files          22       25       +3
  Lines         349      390      +41
==========================================
+ Hits          326      353      +27
- Misses         23       37      +14

Flag
Coverage Δ

linux
90.51% <86.77%> (-2.90%)
:arrow_down:

windows
?

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

Impacted Files
Coverage Δ

otelib/backends/services/__init__.py
100.00% <ø> (ø)

otelib/backends/services/dataresource.py
100.00% <ø> (ø)

otelib/backends/services/filter.py
100.00% <ø> (ø)

otelib/backends/services/function.py
100.00% <ø> (ø)

otelib/backends/services/mapping.py
100.00% <ø> (ø)

otelib/backends/services/transformation.py
100.00% <ø> (ø)

otelib/exceptions.py
78.94% <66.66%> (-21.06%)
:arrow_down:

otelib/backends/factories.py
68.96% <68.96%> (ø)

otelib/backends/python/base.py
76.36% <73.33%> (-13.35%)
:arrow_down:

otelib/backends/python/dataresource.py
81.81% <86.66%> (+5.62%)
:arrow_up:

... and 8 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

"Looks Good"^TM. Unfortunately its just a lot of code for me to digest right now. My schedule is pretty packed the next few days but I could book an hour to look at it in more detail on Wednesday. I am otherwise OK to merge it now, but its just too much for me to think deeply about at the moment.

"Looks Good"^TM. Unfortunately its just a lot of code for me to digest right now. My schedule is pretty packed the next few days but I could book an hour to look at it in more detail on Wednesday. I am otherwise OK to merge it now, but its just too much for me to think deeply about at the moment.

I can't merge until someone approves it.

@CasperWA am I able to approve? In the 'review required' section it only shows @quaat as a reviewer

@CasperWA am I able to approve? In the 'review required' section it only shows @quaat as a reviewer

It just shows him in the list of "1 pending reviewer", because he hasn't reviewed it yet. But you have. You should be able to approve by doing another review normally.


Group Number for task 1

Hi all,
Have you checked on the task 1 of homework 6. It says group number in the range of interval. Are we supposed to use our homework group number or some other number there?

That is a bit confusing. But I don't think that is homework group number. @Pranjal-Srivastava-2023 is doing task 1. Maybe he knows something.

I think it is the homework group number as well, I had this same issue on my project https://github.com/ubsuny/23-Homework6G1/issues/11

Yes, This is our homework group number.


Type Enum rather then string in state

It would be very useful if we replaced the state.Type of lights with a enum rather then a string.
something like
[Flags]
enum LightTypes {
	[Description("on/off light")]
	OnOffLight = 1,
	[Description("dimmable light")]
	DimmableLight = 2,
	[Description("color Temperature Ligh")]
	TempatureLight = 4,
	[Description("color light")]
	ColorLight = 8,
	[Description("extended color light")]
	ExtendedColorLight = 16,

	//Extended Types
	OnOff = OnOffLight | DimmableLight | TempatureLight | ColorLight | ExtendedColorLight,
	Dimmable = DimmableLight | TempatureLight | ColorLight| ExtendedColorLight,
	Tempature = TempatureLight | ColorLight| ExtendedColorLight,
	RGB = ColorLight | ExtendedColorLight
}

and you could parse the incoming values like this
public static T GetEnum<T>(this string description)
{
	Type type = typeof(T);
	if (!type.IsEnum) throw new InvalidOperationException();
	foreach (FieldInfo field in type.GetFields())
	{
		if (Attribute.GetCustomAttribute(field,
			typeof(DescriptionAttribute)) is DescriptionAttribute attribute)
		{
			if (attribute.Description.Equals(description, StringComparison.OrdinalIgnoreCase))
				return (T)field.GetValue(null);
		}
		else
		{
			if (field.Name.Equals(description, StringComparison.OrdinalIgnoreCase))
				return (T)field.GetValue(null);
		}
	}
	return default(T);
}

Yes, sounds good. But are these the only possible values of the light.type field? The documentation has no specification and it might change and break the library in the future.
Maybe it's better to create an extra property, so we keep the raw string value, and an extra Unknown enum value for unknown types?
Are you able to make a Pull Request? That would be awesome.
Thanks!

I think it's best to create an extension method for this.


bug in samples

At least this sample is on an https page failing to load JQuery over http - and failing due to mixed content.
https://sunnywalker.github.io/jQuery.FilterTable/examples/filtertable-existing-input.html

I had updated the examples in the repo, but forgot to fix them in the GitHub pages as well. They're working now, thank you.


Small typo in documentation: probabilitiies should be probabilities

https://github.com/jeremymcrae/bgen/blob/d713890ff13f34528dd0d05436f91a80fa8e3fd7/README.md?plain=1#L95

thanks! fixed in readme now